Skip to content

Commit

Permalink
Use real preprocessing pipeline in unit tests.
Browse files Browse the repository at this point in the history
Tests previously used a mock pipeline because no preprocessing tasks
had been implemented, and empty pipelines are invalid.
  • Loading branch information
kfindeisen committed Sep 16, 2024
1 parent cd37317 commit 2e770c0
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 63 deletions.
49 changes: 0 additions & 49 deletions python/dummyTask.py

This file was deleted.

4 changes: 2 additions & 2 deletions tests/data/ApPipe.yaml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
description: Test version of the full AP pipeline
description: Test version of the main AP pipeline

imports:
- location: $AP_PIPE_DIR/pipelines/LSSTComCamSim/ApPipe.yaml
- location: $PROMPT_PROCESSING_DIR/pipelines/LSSTComCamSim/ApPipe.yaml
parameters:
# Use dataset's specific templates
coaddName: goodSeeing
Expand Down
10 changes: 6 additions & 4 deletions tests/data/MinPrep.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
description: A preprocessing pipeline for testing. Must generate quantum graph, but no tasks actually run.
tasks:
# TODO: replace after DM-31833 or DM-42576
dummy: dummyTask.DummyTask
description: Test pipeline consisting only of APDB preload

imports:
- location: $PROMPT_PROCESSING_DIR/tests/data/Preprocess.yaml
include:
- loadDiaCatalogs
10 changes: 6 additions & 4 deletions tests/data/NoPrep.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
description: A preprocessing pipeline for testing. Must generate quantum graph, but no tasks actually run.
tasks:
# TODO: replace after DM-31833 or DM-42576
dummy: dummyTask.DummyTask
description: Test pipeline consisting only of APDB preload

imports:
- location: $PROMPT_PROCESSING_DIR/tests/data/Preprocess.yaml
include:
- loadDiaCatalogs
8 changes: 4 additions & 4 deletions tests/data/Preprocess.yaml
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
description: A preprocessing pipeline for testing. Must generate quantum graph, but no tasks actually run.
tasks:
# TODO: replace after DM-31833 or DM-42576
dummy: dummyTask.DummyTask
description: Test version of the preprocessing pipeline

imports:
- location: $PROMPT_PROCESSING_DIR/pipelines/LSSTComCamSim/Preprocessing.yaml

0 comments on commit 2e770c0

Please sign in to comment.