test: Potential fixes for 2 code quality findings in tests/end_to_end.rs (#624)

* Apply suggested fix to tests/end_to_end.rs from Copilot Autofix

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>

* Apply suggested fix to tests/end_to_end.rs from Copilot Autofix

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>

* fmt and comment

---------

Signed-off-by: Chris Hennick <4961925+Pr0methean@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Co-authored-by: n4n5 <its.just.n4n5@gmail.com>

Author: 3 people

src/extra_fields/mod.rs

@@ -83,7 +83,6 @@ impl TryFrom<u16> for UsedExtraField {
         )
     }
 }
-// AE-x encryption structure
 
 /// Known Extra fields (PKWARE and Third party) mappings
 pub const EXTRA_FIELD_MAPPING: [u16; 58] = [

tests/end_to_end.rs

@@ -270,13 +270,13 @@ const COPY_ENTRY_NAME: &str = "test/lorem_ipsum_renamed.txt";
 const INTERNAL_COPY_ENTRY_NAME: &str = "test/lorem_ipsum_copied.txt";
 
 #[test]
-fn test_extra_field_access() {
+fn test_extra_field_mapping_constants() {
     // just a test to access the variable in the crate
     use zip::extra_fields::EXTRA_FIELD_MAPPING;
 
-    assert_eq!(EXTRA_FIELD_MAPPING[0], 1);
-    assert_eq!(EXTRA_FIELD_MAPPING[0], 0x0001); // ZIP64 extended information extra field
+    // ZIP64 extended information extra field - 0x0001 which is 1
+    assert_eq!(EXTRA_FIELD_MAPPING[0], 0x0001);
 
-    assert_eq!(EXTRA_FIELD_MAPPING[12], 23);
-    assert_eq!(EXTRA_FIELD_MAPPING[12], 0x0017); // Strong Encryption Header
+    // Strong Encryption Header - 0x0017 which is 23
+    assert_eq!(EXTRA_FIELD_MAPPING[12], 0x0017);
 }