Skip to content

fix: syntax errors in openclaw-plugin test files#144

Merged
jverdicc merged 1 commit intomainfrom
codex/fix-syntax-errors-in-test-files
Mar 2, 2026
Merged

fix: syntax errors in openclaw-plugin test files#144
jverdicc merged 1 commit intomainfrom
codex/fix-syntax-errors-in-test-files

Conversation

@jverdicc
Copy link
Owner

@jverdicc jverdicc commented Mar 2, 2026

Motivation

  • Tests in integrations/openclaw-plugin/tests failed to parse due to JavaScript syntax errors that prevented the test suite from running.
  • These parsing issues blocked CI and local test validation, so the test files needed minimal syntactic fixes to restore correctness.

Description

  • Added a missing catch (err) { throw err; } handler to the try block in integrations/openclaw-plugin/tests/plugin.e2e-daemon.test.js to satisfy the parser's requirement for try/catch/finally pairing.
  • Removed a conflicting same-scope redeclaration and fixed the test block boundaries in integrations/openclaw-plugin/tests/plugin.test.js so plugin is no longer declared twice and the test closures are balanced.
  • Ensured both edits are minimal and preserve the original test intent and assertions.

Testing

  • Ran cd integrations/openclaw-plugin && npm ci which completed successfully.
  • Ran cd integrations/openclaw-plugin && npm run check and npm run build, both completed without errors.
  • Ran the test suite with cd integrations/openclaw-plugin && npm test and the tests passed (final combined run cd integrations/openclaw-plugin && npm ci && npm run check && npm run build && npm test returned EXIT:0).

Codex Task

@jverdicc jverdicc merged commit 89ea1ff into main Mar 2, 2026
4 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant