Skip to content

Commit

Permalink
fix: standardize warning logs and test suite descriptions
Browse files Browse the repository at this point in the history
Updated the console method in ReporterUtils to use `console.warn` for warning messages, ensuring the correct log level is used. Test suite descriptions were also standardized for consistency across test files, enhancing readability and maintainability of the test codebase.
  • Loading branch information
dikkadev committed Nov 17, 2023
1 parent 540c391 commit da217b1
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 9 deletions.
2 changes: 1 addition & 1 deletion src/ReporterUtils.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ const info = (message) => {
*/
const warn = (message) => {
// eslint-disable-next-line no-console
console.info(`[${chalk.yellow(pkg.name)}] ${message}`);
console.warn(`[${chalk.yellow(pkg.name)}] ${message}`);
};

/**
Expand Down
4 changes: 1 addition & 3 deletions test/cypress/component/MyEmptyComponent.spec.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
const specTitle = require("../../../specTitle");

describe(specTitle("The root suite"), () => {
describe("An empty root suite", () => {

});
4 changes: 1 addition & 3 deletions test/cypress/integration/Empty.spec.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
const specTitle = require("../../../specTitle");

describe(specTitle("Empty suite"), () => {
describe("An empty root suite", () => {

});
4 changes: 2 additions & 2 deletions test/specs/Reporter.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -313,7 +313,7 @@ describe('Testing reporter', () => {

});

describe('empty test', () => {
describe('empty suite', () => {

const conditionalTest = isCypressVersionAtLeast(6, 2) ? test : test.skip;
const testDir = path.resolve(testOuputDir, 'empty');
Expand Down Expand Up @@ -373,7 +373,7 @@ describe('Testing reporter', () => {

});

describe('empty component test', () => {
describe('empty component suite', () => {

const conditionalTest = isCypressVersionAtLeast(11, 0) ? test : test.skip;
const testDir = path.resolve(testOuputDir, 'MyEmptyComponent');
Expand Down

0 comments on commit da217b1

Please sign in to comment.