Skip to content

test: fix NUTs @W-22293348@#408

Merged
WillieRuemmele merged 2 commits intomainfrom
er/fix-nuts
Apr 30, 2026
Merged

test: fix NUTs @W-22293348@#408
WillieRuemmele merged 2 commits intomainfrom
er/fix-nuts

Conversation

@EstebanRomero84
Copy link
Copy Markdown
Contributor

What does this PR do?

In this PR 398 we introduced a change that broke some NUTs. When the original PR was merged, these legitimate failures were masked by a server issues that made all test to failed. Now that the environment is stable, I've identified and resolved the underlying issues in the test suite.

The original change removed try/catch block that was unnecessarily altering error message handling. By removing this, the error output now aligns with error handling used across other commands.

What issues does this PR fix or reference?

@W-22293348@

Copy link
Copy Markdown
Contributor

@WillieRuemmele WillieRuemmele left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lGTM, could we add an error codes section to publish, like

  public static readonly errorCodes = toHelpSection('ERROR CODES', {
    'Succeeded (0)': 'Agent published successfully without errors.',
    'Failed (1)': 'Compilation errors found in the Agent Script file.',
  });

@WillieRuemmele WillieRuemmele merged commit 9a00b01 into main Apr 30, 2026
13 checks passed
@WillieRuemmele WillieRuemmele deleted the er/fix-nuts branch April 30, 2026 16:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants