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

feat: add test reporting to component generation #94

Merged
merged 2 commits into from
Jul 30, 2024

Conversation

devpow112
Copy link
Contributor

@devpow112 devpow112 commented Jul 30, 2024

https://desire2learn.atlassian.net/browse/QE-264

This adds the configuration pieces for test reporting and fixes a few other minor inconsistencies.

@devpow112 devpow112 changed the title Add test reporting to component generation feat: add test reporting to component generation Jul 30, 2024
@devpow112 devpow112 force-pushed the depowell/test-reporting branch 2 times, most recently from 760cab1 to 5fe2e0e Compare July 30, 2024 18:51
@@ -1,7 +1,7 @@
import '../<%= hyphenatedName %>.js';
import { expect, fixture, html } from '@brightspace-ui/testing';

describe('<%= className %>', () => {
describe('<%= tagName %>', () => {
Copy link
Contributor Author

@devpow112 devpow112 Jul 30, 2024

Choose a reason for hiding this comment

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

This didn't match what we tend to do in the wild and also didn't match our vdiff suite names. Changing to be all the same

@@ -1,7 +1,7 @@
import '../<%= hyphenatedName %>.js';
import { runConstructor } from '@brightspace-ui/testing';

describe('<%= className %>', () => {
describe('<%= tagName %>', () => {
Copy link
Contributor Author

@devpow112 devpow112 Jul 30, 2024

Choose a reason for hiding this comment

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

This didn't match what we tend to do in the wild and also didn't match our vdiff suite names. Changing to be all the same

Base automatically changed from depowell/axe to main July 30, 2024 19:00
@devpow112 devpow112 force-pushed the depowell/test-reporting branch from 5fe2e0e to 3788c79 Compare July 30, 2024 19:19
@devpow112 devpow112 force-pushed the depowell/test-reporting branch from 3788c79 to 90ed8b2 Compare July 30, 2024 19:20
@devpow112 devpow112 marked this pull request as ready for review July 30, 2024 19:21
@devpow112 devpow112 requested a review from a team as a code owner July 30, 2024 19:21
@devpow112 devpow112 requested a review from a team July 30, 2024 19:21
README.md Outdated Show resolved Hide resolved
src/create.js Outdated Show resolved Hide resolved
src/create.js Outdated Show resolved Hide resolved
Co-authored-by: Dave Lockhart <[email protected]>
@devpow112 devpow112 merged commit 4042c79 into main Jul 30, 2024
1 check passed
@devpow112 devpow112 deleted the depowell/test-reporting branch July 30, 2024 20:33
Copy link

🎉 This PR is included in version 3.3.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

2 participants