Skip to content

Conversation

@seer-by-sentry
Copy link
Contributor

This PR adds tests for #3811

Commits:

  • Add unit tests for Topic classes in space.ts

@seer-by-sentry
Copy link
Contributor Author

✅ Sentry found no issues in your recent changes ✅

@codecov-qa
Copy link

codecov-qa bot commented Mar 21, 2025

❌ 11 Tests Failed:

Tests completed Failed Passed Skipped
4894 11 4883 2
View the top 3 failed test(s) by shortest run time
src/services/codecovAI/space.test.ts > Topic classes > CookingTopic > has the correct name
Stack Traces | 0.000343s run time
TypeError: CookingTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:35:22
src/services/codecovAI/space.test.ts > Topic classes > SpaceTopic > returns a fact about space
Stack Traces | 0.000363s run time
TypeError: SpaceTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:8:20
src/services/codecovAI/space.test.ts > Topic classes > CookingTopic > has the correct id
Stack Traces | 0.000441s run time
TypeError: CookingTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:35:22

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@codecov-public-qa
Copy link

❌ 10 Tests Failed:

Tests completed Failed Passed Skipped
4248 10 4238 0
View the top 3 failed tests by shortest run time
src/services/codecovAI/space.test.ts > Topic classes > CookingTopic > has the correct name
Stack Traces | 0.000343s run time
TypeError: CookingTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:35:22
src/services/codecovAI/space.test.ts > Topic classes > SpaceTopic > returns a fact about space
Stack Traces | 0.000363s run time
TypeError: SpaceTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:8:20
src/services/codecovAI/space.test.ts > Topic classes > CookingTopic > has the correct id
Stack Traces | 0.000441s run time
TypeError: CookingTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:35:22

To view more test analytics, go to the Test Analytics Dashboard
📢 Thoughts on this report? Let us know!

@codecov
Copy link

codecov bot commented Mar 21, 2025

❌ 11 Tests Failed:

Tests completed Failed Passed Skipped
5710 11 5699 3
View the top 3 failed test(s) by shortest run time
src/services/codecovAI/space.test.ts > Topic classes > CookingTopic > has the correct name
Stack Traces | 0.000343s run time
TypeError: CookingTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:35:22
src/services/codecovAI/space.test.ts > Topic classes > SpaceTopic > returns a fact about space
Stack Traces | 0.000363s run time
TypeError: SpaceTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:8:20
src/services/codecovAI/space.test.ts > Topic classes > CookingTopic > has the correct id
Stack Traces | 0.000441s run time
TypeError: CookingTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:35:22

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@codecov-staging
Copy link

❌ 11 Tests Failed:

Tests completed Failed Passed Skipped
4894 11 4883 2
View the top 3 failed test(s) by shortest run time
src/services/codecovAI/space.test.ts > Topic classes > CookingTopic > has the correct name
Stack Traces | 0.000343s run time
TypeError: CookingTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:35:22
src/services/codecovAI/space.test.ts > Topic classes > SpaceTopic > returns a fact about space
Stack Traces | 0.000363s run time
TypeError: SpaceTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:8:20
src/services/codecovAI/space.test.ts > Topic classes > CookingTopic > has the correct id
Stack Traces | 0.000441s run time
TypeError: CookingTopic is not a constructor
 ❯ .../services/codecovAI/space.test.ts:35:22

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

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.

2 participants