Skip to content

Commit ed57a40

Browse files
Darshan808Copilot
andauthored
Apply suggestions from code review
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
1 parent 96e8f5d commit ed57a40

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

tests/test_tpl.py

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -101,13 +101,13 @@ def assert_watch_rebuilds(extension_folder):
101101
# Wait until the watch process is running and its initial build has
102102
# landed in the static directory, so that the comment below is only
103103
# added after watching has started.
104-
wait_for(
104+
assert wait_for(
105105
lambda: (
106106
watch_process.poll() is not None
107107
or list_files_in_static(static_dir) != initial_files
108108
),
109109
timeout=WATCH_INITIAL_BUILD_TIMEOUT,
110-
)
110+
), "Timed out waiting for the initial watch build to complete!"
111111
assert watch_process.poll() is None, "Watch process exited before the initial build!"
112112
files_after_initial_build = list_files_in_static(static_dir)
113113

@@ -125,6 +125,11 @@ def assert_watch_rebuilds(extension_folder):
125125
# List filenames in static directory after change
126126
final_files = list_files_in_static(static_dir)
127127

128+
assert files_after_initial_build != final_files, (
129+
"No changes detected in the static directory after modifying the source."
130+
" Watch process may not have triggered correctly!"
131+
)
132+
128133
# Compare the initial and final file lists
129134
assert initial_files != final_files, (
130135
" No changes detected in the static directory."

0 commit comments

Comments
 (0)