From 9e5a997ec9532e3d3b309536639a146d286e5d92 Mon Sep 17 00:00:00 2001 From: "shigeru.nakajima" Date: Sun, 24 Nov 2024 23:23:40 +0900 Subject: [PATCH] Remove a patch of the test-unit gem. The following pull request has been made to the test-unit gem itself, which will make this patch unnecessary. https://github.com/test-unit/test-unit/pull/262 --- .../ruby-wasm-wasi/test/unit/test_error.rb | 2 -- .../ruby-wasm-wasi/test/unit/test_float.rb | 2 -- .../ruby-wasm-wasi/tools/run-test-unit.mjs | 13 ------------- 3 files changed, 17 deletions(-) diff --git a/packages/npm-packages/ruby-wasm-wasi/test/unit/test_error.rb b/packages/npm-packages/ruby-wasm-wasi/test/unit/test_error.rb index dfec73599e..1438a552a4 100644 --- a/packages/npm-packages/ruby-wasm-wasi/test/unit/test_error.rb +++ b/packages/npm-packages/ruby-wasm-wasi/test/unit/test_error.rb @@ -2,8 +2,6 @@ require "js" class JS::TestError < Test::Unit::TestCase - using JsObjectTestable - def test_throw_error e = assert_raise(JS::Error) { JS.eval("throw new Error('foo')") } assert_match /^Error: foo/, e.message diff --git a/packages/npm-packages/ruby-wasm-wasi/test/unit/test_float.rb b/packages/npm-packages/ruby-wasm-wasi/test/unit/test_float.rb index 4f6df081de..917bca56f4 100644 --- a/packages/npm-packages/ruby-wasm-wasi/test/unit/test_float.rb +++ b/packages/npm-packages/ruby-wasm-wasi/test/unit/test_float.rb @@ -2,8 +2,6 @@ require "js" class JS::TestFloat < Test::Unit::TestCase - using JsObjectTestable - def test_to_js assert_equal (1.0).to_js, JS.eval("return 1.0;") assert_equal (0.5).to_js, JS.eval("return 0.5;") diff --git a/packages/npm-packages/ruby-wasm-wasi/tools/run-test-unit.mjs b/packages/npm-packages/ruby-wasm-wasi/tools/run-test-unit.mjs index 41c850a32d..ffef02c7dd 100755 --- a/packages/npm-packages/ruby-wasm-wasi/tools/run-test-unit.mjs +++ b/packages/npm-packages/ruby-wasm-wasi/tools/run-test-unit.mjs @@ -166,22 +166,9 @@ const test = async (instantiate) => { const rootTestFile = "/__root__/test/test_unit.rb"; const { vm } = await instantiate(rootTestFile); - await vm.evalAsync(` require 'test/unit' - # FIXME: This is a workaround for the test-unit gem. - # It will be removed when the next pull request is merged and released. - # https://github.com/test-unit/test-unit/pull/262 - require 'pp' - module JsObjectTestable - refine JS::Object do - [:object_id, :pretty_inspect].each do |method| - define_method(method, ::Object.instance_method(method)) - end - end - end - require_relative '${rootTestFile}' ok = Test::Unit::AutoRunner.run exit(1) unless ok