fix(color): off-by-one for INDEXED_COLORS access

src/structs/color.rs

@@ -100,7 +100,7 @@ impl Color {
     /// In that case, use get_argb_with_theme(&self, theme: &Theme).
     pub fn get_argb(&self) -> &str {
         if self.indexed.has_value() {
-            match INDEXED_COLORS.get(self.indexed.get_value().clone() as usize - 1) {
+            match INDEXED_COLORS.get(self.indexed.get_value().clone() as usize) {
                 Some(v) => return v,
                 None => {}
             }
@@ -138,7 +138,7 @@ impl Color {
         let indexed = INDEXED_COLORS.iter().position(|&r| r == argb);
         match indexed {
             Some(v) => {
-                self.indexed.set_value((v + 1) as u32);
+                self.indexed.set_value(v as u32);
                 self.argb.remove_value();
             }
             None => {
@@ -309,7 +309,7 @@ mod tests {
 
         let mut obj = Color::default();
         obj.set_argb("FFFF8080");
-        assert_eq!(obj.get_indexed(), &22);
+        assert_eq!(obj.get_indexed(), &21);
         assert_eq!(obj.get_argb(), "FFFF8080");
 
         let mut obj = Color::default();

tests/integration_test.rs

@@ -1400,3 +1400,15 @@ fn issue_188() {
     let path = std::path::Path::new("./tests/result_files/issue_188.xlsx");
     let _ = umya_spreadsheet::writer::xlsx::write(&book, path);
 }
+
+#[test]
+fn expect_red_indexed_color() {
+    let path = std::path::Path::new("./tests/test_files/red_indexed_color.xlsx");
+    let book = umya_spreadsheet::reader::xlsx::read(path).unwrap();
+
+    let cell = book.get_sheet(&0).unwrap().get_cell("A1").unwrap();
+
+    let color = cell.get_style().get_font().unwrap().get_color().get_argb();
+
+    assert_eq!("FFFF0000", color);
+}