Skip to content
Snippets Groups Projects

lose_meaningless_tests

Merged lose_meaningless_tests
lose_meaningless_tests into main
1 unresolved thread
Merged Janet Goldstein requested to merge lose_meaningless_tests into main
1 unresolved thread
  • get rid of the tests that didn't actually test anything
  • fix directory structure so it's consistent with rest of codebase

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
apps/cli/executables/pexable/ingest/tests/unit/test_ingestion_manifest.py → apps/cli/executables/pexable/ingest/test/test_ingestion_manifest.py
15 15 #
16 16 # You should have received a copy of the GNU General Public License
17 17 # along with Workspaces. If not, see <https://www.gnu.org/licenses/>.
18
19 """ Tests of the ingestion manifest """
18 20 from pathlib import Path
19 21
22 # pylint: disable=C0116, E0401, W0621
23
  • Comment on lines +22 to +23
    Nathan Hertz
    Nathan Hertz @nhertz ·

    Can you put this pylint ignore (and all future ones) above all the imports so Black won't try to move them like it always wants to do?

  • Please register or sign in to reply
  • Nathan Hertz approved this merge request

    approved this merge request

  • Janet Goldstein added 1 commit

    added 1 commit

    • e0fea54d - lose_meaningless_tests: move pylint ignores up top

    Compare with previous version

  • Charlotte Hausman approved this merge request

    approved this merge request

  • Janet Goldstein added 5 commits

    added 5 commits

    • 7085f19f - 1 commit from branch main
    • d4e0b50e - lose_meaningless_tests: get rid of the tests that didn't actually test...
    • 613a7682 - lose_meaningless_tests: get rid of the tests that didn't actually test...
    • 5e19c1f6 - lose_meaningless_tests: move pylint ignores up top
    • 695ff17a - Merge branch 'lose_meaningless_tests' of...

    Compare with previous version

    Toggle commit list
  • Janet Goldstein merged

    merged

    • Please register or sign in to reply