Skip to content

Commit 5b0badc

Browse files
authored
Merge pull request #59 from PennChopMicrobiomeProgram/58-safeguard-against-test-db-overwrites
Require test db to be empty for creation
2 parents 0a71ab9 + a5f471a commit 5b0badc

File tree

1 file changed

+4
-8
lines changed
  • app/metadatalib/src/metadatalib

1 file changed

+4
-8
lines changed

app/metadatalib/src/metadatalib/db.py

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,15 +14,11 @@ def create_test_db(session: Session = None):
1414
session = imported_session
1515
Base.metadata.create_all(engine)
1616

17-
if "sqlite" not in SQLALCHEMY_DATABASE_URI:
18-
print("Not a SQLite database, skipping test database creation.")
19-
sys.exit(1)
20-
2117
if session.query(Project).count():
22-
session.execute(delete(Project))
23-
session.execute(delete(Submission))
24-
session.execute(delete(Sample))
25-
session.execute(delete(Annotation))
18+
sys.stderr.write(
19+
"Database already contains data, skipping test data creation\n"
20+
)
21+
sys.exit(1)
2622

2723
p1 = Project(
2824
project_id=1,

0 commit comments

Comments
 (0)