From 7aaeac03d622eaef670611a614268836bfe4f50f Mon Sep 17 00:00:00 2001 From: Sebastian Serth Date: Wed, 16 Nov 2022 22:41:07 +0100 Subject: [PATCH] Remove outdated error for used_defined_tests Previously, a user-defined test (those run with "Test" instead of "Run") would cause a flash message and a Sentry error if anything was printed to StdErr. As this might happen during user code execution (and therefore is no error), we remove this warning. All existing errors are already caught (compare with enum status in testrun.rb), so it's fine for now. Fixes CODEOCEAN-BT --- app/assets/javascripts/editor/editor.js.erb | 6 ------ 1 file changed, 6 deletions(-) diff --git a/app/assets/javascripts/editor/editor.js.erb b/app/assets/javascripts/editor/editor.js.erb index a444bac8..13018eea 100644 --- a/app/assets/javascripts/editor/editor.js.erb +++ b/app/assets/javascripts/editor/editor.js.erb @@ -705,12 +705,6 @@ var CodeOceanEditor = { this.showContainerDepletedMessage(); } else if (output.status === 'out_of_memory') { this.showOutOfMemoryMessage(); - } else if (output.stderr) { - $.flash.danger({ - icon: ['fa-solid', 'fa-bug'], - text: $('#run').data('message-failure') - }); - Sentry.captureException(JSON.stringify(output)); } },