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

Add input capture for allure reporting #135

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

PatStLouis
Copy link
Collaborator

This will capture requests and response sent to the issuer/verifier for every call made, useful for debugging.

To avoid this creating unnecessary noise when running with the db reporter, the console logging is disabled when calling the attachment function.

Copy link
Collaborator

@tminard tminard left a comment

Choose a reason for hiding this comment

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

I like it. Can you add info to readme on how to use it?

@BigBlueHat
Copy link
Member

I like it. Can you add info to readme on how to use it?

As long as #134 goes in first. 😁

@PatStLouis is Allure already configured for the weekly report runs? What happens if Allure is turned off or not configured (or goes missing / is down)?

@PatStLouis
Copy link
Collaborator Author

@tminard there is a section about running allure, you just need to initialize the tests with
npx mocha tests/ then build the report with npx allure serve

The rest is automagical, just need to browse the report and the payload will be attached to their associated tests.

@PatStLouis
Copy link
Collaborator Author

@BigBlueHat currently this is all configured to be run locally, no standing allure server is setup. I can set one up if we want to explore this further.

The way its currently implemented, a user runs the test locally and runs the allure server from the command line The report will be available from the local host.

export async function addJsonAttachment(fileName, content) {
try {
// Temporarily disable the console log to avoid unnecessary info logs.
const consoleLog = console.log;
Copy link
Contributor

Choose a reason for hiding this comment

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

@PatStLouis so not sure if we're all 100% agreeing on how to solve this, but do you think this will work:

  1. create a separate command for allure reports
  2. that separate command can turn off the consoleLog if needed

LMK if that will work.

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.

4 participants