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

Tests for TextArea, Administrator, and VM User Answer Tagging #1989

Merged
merged 10 commits into from
Jun 7, 2021
Merged

Tests for TextArea, Administrator, and VM User Answer Tagging #1989

merged 10 commits into from
Jun 7, 2021

Conversation

johnbumgardner
Copy link
Collaborator

Please set the title of your pull request in the following format: [Project id/Independent study/Issue fix/...]. project name, eg., E904. Integrating courses and assignments on Expertiza with Moodle

We suggest you including the below information in your pull request:

  • A description of the changes proposed in the pull request.
  • @mentions of the person or team responsible for reviewing proposed changes.

About Expertiza Bot

  • If you have any questions about comments given by the expertiza-bot (example), you could create a comment in your pull request with the format /dispute [UUID1] [UUID2] (example) then the professor and TAs will be notified.
  • Here is a video about how to communicate to expertiza-bot.

@expertiza-bot
Copy link
Contributor

1 Message
💬 Thanks for the pull request, and welcome! 🎉 The Expertiza team is excited to review your changes, and you should hear from us soon.

This repository is being automatically checked for code quality issues using Code Climate.
You can see results for this analysis in the PR status below. Newly introduced issues should be fixed before a pull request is considered ready to review.

If you have any questions, please send email to expertiza-support@lists.ncsu.edu.

UUID 2 Warnings
2ec0 ⚠️ Using create in unit tests or integration tests may be overkill. Try to use build or double instead.
3ecd ⚠️ You modified spec/factories/ folder; please double-check whether it is necessary.

Generated by expertiza-bot

@coveralls
Copy link

Coverage Status

Coverage decreased (-28.5%) to 37.175% when pulling 7b75abd on johnbumgardner:more-tests-3 into b93abf6 on expertiza:beta.

@coveralls
Copy link

coveralls commented Jun 7, 2021

Coverage Status

Coverage decreased (-9.9%) to 55.741% when pulling 7b75abd on johnbumgardner:more-tests-3 into b93abf6 on expertiza:beta.

@expertiza-travisci-bot
Copy link

Hey @johnbumgardner,
Your changes look good to me! 🎉

@johnbumgardner johnbumgardner merged commit aab69a1 into expertiza:beta Jun 7, 2021
@johnbumgardner johnbumgardner deleted the more-tests-3 branch September 1, 2021 23:44
@duhaoze11 duhaoze11 mentioned this pull request Oct 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

None yet

4 participants