Enhance CLI test functionality and improve error messages in verification process

Author: zombocoder

.github/workflows/release.yml

@@ -71,35 +71,92 @@ jobs:
           cd build
           ctest --output-on-failure
 
-      - name: Test CLI
+      - name: Test CLI functionality
         run: |
           # Create test data
           mkdir -p test_data
-          echo "Release test" > test_data/test.txt
-          echo "Sensitive data for encryption test" > test_data/secret.txt
+          echo "Hello World" > test_data/hello.txt
+          echo "Goodbye" > test_data/bye.txt
+          mkdir -p test_data/subdir
+          echo "Nested file" > test_data/subdir/nested.txt
 
-          # Test basic functionality
+          # Test CLI workflow
           ./build/bin/bfc create test.bfc test_data/
           ./build/bin/bfc list test.bfc
+          ./build/bin/bfc info test.bfc
           ./build/bin/bfc verify test.bfc
-
-          # Test encryption functionality (password-based)
-          ./build/bin/bfc create -e testpass encrypted.bfc test_data/
-          ./build/bin/bfc list encrypted.bfc
-          ./build/bin/bfc verify encrypted.bfc
-          
-          # Test extraction with password
-          mkdir -p extracted
-          ./build/bin/bfc extract -p testpass -C extracted encrypted.bfc
-          diff test_data/test.txt extracted/test.txt
-          diff test_data/secret.txt extracted/secret.txt
+          ./build/bin/bfc verify --deep test.bfc
+
+          # Test compression functionality
+          ./build/bin/bfc create -c zstd test_compressed.bfc test_data/
+          ./build/bin/bfc info test_compressed.bfc test_data/hello.txt
+          ./build/bin/bfc verify test_compressed.bfc
+
+          # Test different compression levels
+          ./build/bin/bfc create -c zstd -l 1 test_fast.bfc test_data/
+          ./build/bin/bfc create -c zstd -l 6 test_balanced.bfc test_data/
+          ./build/bin/bfc info test_balanced.bfc test_data/hello.txt
+
+          # Test extraction
+          mkdir -p extract_test
+          cd extract_test
+          ../build/bin/bfc extract ../test.bfc
+
+          # Verify extracted files
+          [ -f hello.txt ] && echo "hello.txt extracted"
+          [ -f bye.txt ] && echo "bye.txt extracted"
+          [ -f subdir/nested.txt ] && echo "nested.txt extracted"
+
+          cd ..
+          rm -rf extract_test
+
+          # Test encryption functionality
+          echo "Testing encryption features..."
+          ./build/bin/bfc create -e testpassword123 test_encrypted.bfc test_data/
+          ./build/bin/bfc info test_encrypted.bfc
+          ./build/bin/bfc info test_encrypted.bfc test_data/hello.txt | grep -i encrypt
+
+          # Test encrypted extraction
+          mkdir -p extract_encrypted
+          cd extract_encrypted
+          ../build/bin/bfc extract -p testpassword123 ../test_encrypted.bfc
+
+          # Verify extracted files from encrypted container
+          [ -f hello.txt ] && echo "hello.txt extracted from encrypted container"
+          [ -f bye.txt ] && echo "bye.txt extracted from encrypted container"
+          [ -f subdir/nested.txt ] && echo "nested.txt extracted from encrypted container"
+
+          cd ..
+          rm -rf extract_encrypted
+
+          # Test wrong password failure
+          mkdir -p extract_fail_test
+          cd extract_fail_test
+          ! ../build/bin/bfc extract -p wrongpassword ../test_encrypted.bfc && echo "Correctly failed with wrong password"
+          cd ..
+          rm -rf extract_fail_test
+
+          # Test key file encryption
+          echo -n "0123456789abcdef0123456789abcdef" > test.key
+          ./build/bin/bfc create -k test.key test_keyfile.bfc test_data/
+          ./build/bin/bfc info test_keyfile.bfc | grep -i encrypt
+
+          mkdir -p extract_keyfile
+          cd extract_keyfile
+          ../build/bin/bfc extract -K ../test.key ../test_keyfile.bfc
+          [ -f hello.txt ] && echo "hello.txt extracted with key file"
+          cd ..
+          rm -rf extract_keyfile
 
           # Test encryption with compression
-          ./build/bin/bfc create -e testpass -c zstd compressed_encrypted.bfc test_data/
-          ./build/bin/bfc verify compressed_encrypted.bfc
+          echo "This is a larger test file with enough content to be compressed by zstd compression algorithm." > test_data/large.txt
+          ./build/bin/bfc create -e testpass -c zstd test_enc_comp.bfc test_data/
+          ./build/bin/bfc info test_enc_comp.bfc test_data/large.txt | grep -i encrypt
+          ./build/bin/bfc info test_enc_comp.bfc test_data/large.txt | grep -i zstd
 
           # Clean up
-          rm -rf test_data test.bfc encrypted.bfc compressed_encrypted.bfc extracted
+          rm -rf test.bfc test_data test_compressed.bfc test_fast.bfc test_balanced.bfc
+          rm -rf test_encrypted.bfc test_keyfile.bfc test_enc_comp.bfc test.key
 
       - name: Get version
         id: get_version

examples/encrypt_example.c

@@ -480,9 +480,9 @@ static int run_demo(void) {
         unlink("/tmp/test_decrypt");
 
         if (result != BFC_OK) {
-          printf("✓ Correctly failed with wrong password: %d\n", result);
+          printf("Correctly failed with wrong password: %d\n", result);
         } else {
-          printf("✗ WARNING: Decryption succeeded with wrong password!\n");
+          printf("WARNING: Decryption succeeded with wrong password!\n");
         }
       }
     }

src/cli/cmd_verify.c

@@ -187,13 +187,13 @@ int cmd_verify(int argc, char* argv[]) {
       if (opts.show_progress && opts.deep_verify) {
         printf("\n");
       }
-      printf("✓ Verification successful\n");
+      printf("Verification successful\n");
       printf("Container is valid and all checksums match\n");
       printf("Verification completed in %.2f seconds\n", elapsed);
     }
     return 0;
   } else {
-    print_error("✗ Verification failed: %s", bfc_error_string(result));
+    print_error("Verification failed: %s", bfc_error_string(result));
 
     // Provide more specific error information
     switch (result) {