Skip to content

Conversation

james-toussaint
Copy link
Contributor

  • Logs:
14:58:11    Integration tests
14:58:11  Starting services..
[...]
15:04:44      ✔ should get protocol (253ms)
[...]
15:04:44    1 passing (7m)
  • Main & develop are now protected and both unit & integration test checks are mandatory:
    image

Copy link
Member

@zguesmi zguesmi left a comment

Choose a reason for hiding this comment

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

Well done!

run: npm run test
- name: Test build
run: npm run build
# See Jenkinsfile-itest for "Run integration tests" step
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@DamienMure FYI 1

}
}
docker.image('node:22-alpine')
.inside('-v /var/run/docker.sock:/var/run/docker.sock --network=host --user=root') {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@DamienMure FYI 2

@james-toussaint james-toussaint merged commit 60f469d into develop Oct 30, 2024
2 checks passed
@james-toussaint james-toussaint deleted the feature/add-protocol-integration-test branch October 30, 2024 14:33
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.

3 participants