tkmind/tkmind_go
1
0
Fork 0
Code Issues Pull Requests Actions 4 Packages Projects Releases Wiki Activity

correct the tests from an odd merge (#4804)

Browse Source
This commit is contained in:
Michael Neale
2025-09-25 10:07:10 +10:00
committed by GitHub
parent 3fa8599522
commit 05bd4da179
1 changed files with 11 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files
crates/goose-mcp/src/computercontroller/xlsx_tool.rs
+11 -11
View File
@@ -355,12 +355,12 @@ mod tests {
// A2 should be row=2, col=1 // A2 should be row=2, col=1
let a2 = xlsx.get_cell_value(worksheet, 2, 1)?; let a2 = xlsx.get_cell_value(worksheet, 2, 1)?;
println!("A2 (2,1): {}", a2.value); println!("A2 (2,1): {}", a2.value);
assert_eq!(a2.value, "Country"); assert_eq!(a2.value, "Government");
// B1 should be row=1, col=2 // B1 should be row=1, col=2
let b1 = xlsx.get_cell_value(worksheet, 1, 2)?; let b1 = xlsx.get_cell_value(worksheet, 1, 2)?;
println!("B1 (1,2): {}", b1.value); println!("B1 (1,2): {}", b1.value);
assert_eq!(b1.value, "Government"); assert_eq!(b1.value, "Country");
// B2 should be row=2, col=2 // B2 should be row=2, col=2
let b2 = xlsx.get_cell_value(worksheet, 2, 2)?; let b2 = xlsx.get_cell_value(worksheet, 2, 2)?;
@@ -386,14 +386,14 @@ mod tests {
// Test that A2 (row 2, column 1) returns the correct value // Test that A2 (row 2, column 1) returns the correct value
let a2_value = xlsx.get_cell_value(worksheet, 2, 1)?; let a2_value = xlsx.get_cell_value(worksheet, 2, 1)?;
assert_eq!(a2_value.value, "Country", "A2 should contain 'Country'"); assert_eq!(
a2_value.value, "Government",
"A2 should contain 'Government'"
);
// Test that B1 (row 1, column 2) returns its own value, not A2's // Test that B1 (row 1, column 2) returns its own value, not A2's
let b1_value = xlsx.get_cell_value(worksheet, 1, 2)?; let b1_value = xlsx.get_cell_value(worksheet, 1, 2)?;
assert_eq!( assert_eq!(b1_value.value, "Country", "B1 should contain 'Country'");
b1_value.value, "Government",
"B1 should contain 'Government'"
);
// Additional verification with ranges // Additional verification with ranges
let range = xlsx.get_range(worksheet, "A1:B2")?; let range = xlsx.get_range(worksheet, "A1:B2")?;
@@ -402,12 +402,12 @@ mod tests {
"A1 should be 'Segment'" "A1 should be 'Segment'"
); );
assert_eq!( assert_eq!(
range.values[0][1].value, "Government", range.values[0][1].value, "Country",
"B1 should be 'Government'" "B1 should be 'Country'"
); );
assert_eq!( assert_eq!(
range.values[1][0].value, "Country", range.values[1][0].value, "Government",
"A2 should be 'Country'" "A2 should be 'Government'"
); );
assert_eq!(range.values[1][1].value, "Canada", "B2 should be 'Canada'"); assert_eq!(range.values[1][1].value, "Canada", "B2 should be 'Canada'");