Rdf.StrictMode Test Likely Produces Undefined Behaviour (hashtag2867)

[vikramsubramanian]

In the initial CI check for [this]( PR, Rdf.StrictMode failed in gcc and gcc 32 bit tests. The pull request only changes a workflow file and does not touch any other code. It produced this log:

cypherScript: /home/runner/actions-runner/_work/kuzu/kuzu/dataset/EMPTY/schema.cypher
cypherScript: /home/runner/actions-runner/_work/kuzu/kuzu/dataset/EMPTY/schema.cypher doesn't exist. Skipping...
cypherScript: /home/runner/actions-runner/_work/kuzu/kuzu/dataset/EMPTY/copy.cypher
cypherScript: /home/runner/actions-runner/_work/kuzu/kuzu/dataset/EMPTY/copy.cypher doesn't exist. Skipping...
[2024-02-12 17:56:23.019] [info] DEBUG LOG: 
[2024-02-12 17:56:23.019] [info] QUERY: CREATE RDFGraph R;
[2024-02-12 17:56:23.068] [info] DEBUG LOG: 
[2024-02-12 17:56:23.068] [info] QUERY: COPY R FROM "/home/runner/actions-runner/_work/kuzu/kuzu/dataset/rdf/malformed/invalid_syntax.ttl" (in_memory=true, strict=true);
[2024-02-12 17:56:23.072] [info] DEBUG LOG: 
[2024-02-12 17:56:23.072] [info] QUERY: COPY R FROM "/home/runner/actions-runner/_work/kuzu/kuzu/dataset/rdf/malformed/invalid_prefix.ttl" (in_memory=false, strict=true);
[2024-02-12 17:56:23.074] [info] EXPECTED ERROR: Runtime exception: Invalid syntax while reading file at line 4 and col 29
/home/runner/actions-runner/_work/kuzu/kuzu/test/test_runner/test_runner.cpp:29: Failure
Value of: testStatement(statement, conn, databasePath)
  Actual: false
Expected: true
[2024-02-12 17:56:23.074] [info] DEBUG LOG: 
[2024-02-12 17:56:23.074] [info] QUERY: COPY R FROM "/home/runner/actions-runner/_work/kuzu/kuzu/dataset/rdf/malformed/infinite_loop.ttl" (strict=true);

)

[mayil-ai[bot]]

Summary: Rdf.StrictMode test in the CI check likely produces undefined behavior.

Possible Solution

Based on the provided information, the issue seems to be related to the CI check failure in gcc and gcc 32-bit tests after a pull request that only changes a workflow file. The log messages indicate problems with missing cypherScript files and execution of RDF-related queries, as well as a specific error message about invalid syntax in test_runner.cpp.

To address the issue, consider the following steps:

• Verify that the workflow file changes do not inadvertently affect the test environment setup, such as paths to test scripts or required dependencies.
• Ensure that the Rdf.StrictMode is correctly configured in the tests. This mode should enforce strict parsing rules, and any deviation from expected syntax should result in a test failure.
• Check the testStatement function in test_runner.cpp around line 29 to ensure it is correctly handling malformed RDF files when Rdf.StrictMode is enabled.
• Review the test cases to ensure that they are correctly referencing the cypherScript files. If the files are missing or paths are incorrect, correct them or update the test cases to reflect the new locations.
• If the error message "EXPECTED ERROR: Runtime exception: Invalid syntax while reading file at line 4 and col 29" is part of a test case that expects a failure, ensure that the test is correctly asserting this expected failure.
• If the CI system is skipping tests due to missing files, as indicated by the log message "cypherScript: ... doesn't exist. Skipping...", ensure that the necessary files are present or that the test runner is configured to ignore or handle missing files appropriately.
• Review the gcc and gcc 32-bit test configurations in the workflow files to ensure they are set up correctly and are consistent with other successful test environments.

If the issue persists after these checks, further investigation into the test suite and CI configuration may be necessary to identify the root cause of the test failures.

Code snippets to check

• .github/workflows/ci-workflow.yml

  The issue is related to a CI workflow failure, and this file contains the CI workflow definitions that may need to be inspected to understand the cause of the failure.

• .github/workflows/multiplatform-build-test.yml

  This file also contains CI workflow definitions for multiplatform builds and tests, which could be relevant to the issue if the failure is related to platform-specific configurations.

• test/main/csv_output_test.cpp

  The issue mentions a test failure, and this file contains test code that may be related to the Rdf.StrictMode failure.