Skip to content
This repository has been archived by the owner on Dec 1, 2022. It is now read-only.

[Minor] Add job-related pytest #1233

Merged
merged 7 commits into from
Jul 14, 2021
Merged

[Minor] Add job-related pytest #1233

merged 7 commits into from
Jul 14, 2021

Conversation

Aiee
Copy link
Contributor

@Aiee Aiee commented Jul 9, 2021 •
edited
Loading

  1. Fix outdated jobs tests
  2. Check the datatypes returned
  3. Add job-related tests to make test

@Aiee Aiee added the ready-for-testing PR: ready for the CI test label Jul 9, 2021
laura-ding
laura-ding previously approved these changes Jul 9, 2021
View reviewed changes
yixinglu
yixinglu reviewed Jul 12, 2021
View reviewed changes
tests/job/test_jobs.py Outdated Show resolved Hide resolved
@Aiee Aiee force-pushed the add-jobs-pytest branch from a0d6286 to 15e38a9 Compare July 12, 2021 07:46
@yixinglu yixinglu merged commit 937edae into vesoft-inc:master Jul 14, 2021
@Aiee Aiee deleted the add-jobs-pytest branch July 14, 2021 09:07
@jamieliu1023 jamieliu1023 mentioned this pull request Jul 17, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@yixinglu yixinglu yixinglu approved these changes

@jievince jievince jievince approved these changes

@laura-ding laura-ding laura-ding left review comments

@CPWstatic CPWstatic Awaiting requested review from CPWstatic

@czpmango czpmango Awaiting requested review from czpmango

@nevermore3 nevermore3 Awaiting requested review from nevermore3

@Shylock-Hg Shylock-Hg Awaiting requested review from Shylock-Hg

Assignees
No one assigned
Labels
ready-for-testing PR: ready for the CI test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Aiee @yixinglu @jievince @laura-ding