add integ test for default param bug

Author: alex-treebeard

.github/workflows/action_integration_test.yml

@@ -58,6 +58,26 @@ jobs:
           path: "./integration-test/partially_passing"
           notebooks: "main.ipynb"
 
+  partially_passing_2:
+    timeout-minutes: 20
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-python@v2
+      - uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
+        with:
+          project_id: ${{ secrets.GCP_PROJECT_ID }}
+          service_account_key: ${{ secrets.GCP_SA_KEY }}
+          export_default_credentials: true
+      - run: gcloud auth configure-docker
+      - uses: "./"
+        with:
+          api-key: ${{ secrets.TREEBEARD_API_KEY }}
+          docker-registry-prefix: gcr.io/treebeard-259315/tests
+          debug: "true"
+          path: "./integration-test/partially_passing"
+          # notebooks configured in treebeard.yaml
+
   failing_build:
     timeout-minutes: 20
     runs-on: ubuntu-latest

integration-test/partially_passing/treebeard.yaml

@@ -3,3 +3,5 @@ ignore:
   - .vscode
   - .ipynb_checkpoints
   - .pytest_cache
+notebooks:
+  - main.ipynb

treebeard-lib/treebeard/notebooks/commands.py

@@ -164,7 +164,7 @@ def run_repo(
         treebeard_config.notebooks = notebooks
 
     if ignore:
-        treebeard_config.ignore = ignore
+        treebeard_config.ignore = ignore + treebeard_config.output_dirs
 
     if "TREEBEARD_START_TIME" not in os.environ:
         os.environ["TREEBEARD_START_TIME"] = get_time()
@@ -200,13 +200,8 @@ def run_repo(
             update_url=f"{api_url}/{treebeard_context.treebeard_env.run_path}/update",
         )
 
-    if treebeard_config:
-        ignore += treebeard_config.ignore + treebeard_config.output_dirs
-
     click.echo("🌲  Creating Project bundle")
-
     temp_dir = tempfile.mkdtemp()
-
     copy_tree(os.getcwd(), str(temp_dir), preserve_symlinks=1)
 
     # Overwrite config with in-memory-modified