Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ci(workflows): add test and publish workflows #2

Merged
merged 4 commits into from
Mar 5, 2024
Merged

Conversation

steebchen
Copy link
Collaborator

@steebchen steebchen commented Mar 4, 2024

TODO:

  • add NPM_TOKEN to repo settings

@@ -52,7 +52,7 @@ describe('AdminClient', () => {
});

describe('put_workflow', () => {
it('should throw an error if no version and not auto version', async () => {
it.skip('should throw an error if no version and not auto version', async () => {
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

ayoo

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Expected substring: "PutWorkflow error: workflow version is required, or use autoVersion"
Received message:   "/WorkflowService/PutWorkflow UNAVAILABLE: No connection established. Last error: connect ECONNREFUSED 127.0.0.1:50051 (2024-03-05T14:24:08.605Z)"

      53 |    * determine if the workflow definition has changed and create a new version if necessary.
      54 |    * @param workflow a workflow definition to create
    > 55 |    */
         |      ^
      56 |   async put_workflow(workflow: CreateWorkflowVersionOpts) {
      57 |     try {
      58 |       await retrier(async () => this.client.putWorkflow({ opts: workflow }), this.logger);

      at AdminClient.<anonymous> (src/clients/admin/admin-client.ts:55:23)

@steebchen steebchen merged commit 8c844bc into origin/main Mar 5, 2024
1 check passed
@steebchen steebchen deleted the ci/wfs branch March 5, 2024 16:49
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.

1 participant