From 78650e162d4e88fd71ec075f50cc9158eb1dc9d9 Mon Sep 17 00:00:00 2001 From: Neil Fulwiler Date: Fri, 4 Oct 2024 17:39:49 -0400 Subject: [PATCH] use real run uuid in unit test (#25086) ## Summary & Motivation use a UUID4 for a run id instead of "sldkfjsldf" ## How I Tested These Changes ran the unit test Co-authored-by: Neil Fulwiler --- .../dagster/dagster_tests/storage_tests/utils/run_storage.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/python_modules/dagster/dagster_tests/storage_tests/utils/run_storage.py b/python_modules/dagster/dagster_tests/storage_tests/utils/run_storage.py index 7bf13c08054f3..fe90e70b4de24 100644 --- a/python_modules/dagster/dagster_tests/storage_tests/utils/run_storage.py +++ b/python_modules/dagster/dagster_tests/storage_tests/utils/run_storage.py @@ -4,6 +4,7 @@ import unittest from datetime import datetime, timedelta from typing import Optional +from uuid import uuid4 import pytest from dagster import _seven, job, op @@ -1094,7 +1095,7 @@ def test_add_get_snapshot(self, storage): assert not storage.has_job_snapshot(job_snapshot_id) def test_single_write_read_with_snapshot(self, storage: RunStorage): - run_with_snapshot_id = "lkasjdflkjasdf" + run_with_snapshot_id = str(uuid4()) job_def = GraphDefinition(name="some_pipeline", node_defs=[]).to_job() job_snapshot = job_def.get_job_snapshot()