Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor: Migrate to Minitest 5 and ActiveSupport 5 #143

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .env
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
API_URL=http://localhost:9393
ONTOLOGIES_LINKED_DATA_PATH=
GOO_PATH=
SPARQL_CLIENT_PATH=
ONTOPORTAL_KB=ontoportal_kb
REDIS_HOST=redis-ut
15 changes: 9 additions & 6 deletions Gemfile
Original file line number Diff line number Diff line change
@@ -1,11 +1,11 @@
source 'https://rubygems.org'

gem 'activesupport', '~> 3.2'
gem 'activesupport', '~> 5'
# see https://github.com/ncbo/ontologies_api/issues/69
gem 'bigdecimal', '1.4.2'
gem 'bigdecimal'
gem 'faraday', '~> 1.9'
gem 'json-schema', '~> 2.0'
gem 'multi_json', '~> 1.0'
gem 'oj', '~> 3.0'
gem 'multi_json'
gem 'oj'
gem 'parseconfig'
gem 'rack'
gem 'rake', '~> 10.0'
Expand Down Expand Up @@ -69,9 +69,12 @@ group :profiling do
end

group :test do
gem 'minitest', '~> 4.0'
gem 'minitest', '~> 5.0'
gem 'minitest-stub_any_instance'
gem 'rack-test'
gem 'simplecov', require: false
gem 'simplecov-cobertura' # for codecov.io
gem 'crack', '0.4.5'
gem 'webmock'
gem "minitest-hooks", "~> 1.5"
end
82 changes: 55 additions & 27 deletions Gemfile.lock
Original file line number Diff line number Diff line change
Expand Up @@ -100,9 +100,11 @@ GIT
GEM
remote: https://rubygems.org/
specs:
activesupport (3.2.22.5)
i18n (~> 0.6, >= 0.6.4)
multi_json (~> 1.0)
activesupport (5.2.8.1)
concurrent-ruby (~> 1.0, >= 1.0.2)
i18n (>= 0.7, < 2)
minitest (~> 5.1)
tzinfo (~> 1.1)
addressable (2.8.6)
public_suffix (>= 2.0.2, < 6.0)
airbrussh (1.5.1)
Expand All @@ -112,7 +114,7 @@ GEM
base64 (0.2.0)
bcrypt (3.1.20)
bcrypt_pbkdf (1.1.0)
bigdecimal (1.4.2)
bigdecimal (3.1.7)
builder (3.2.4)
capistrano (3.18.1)
airbrussh (>= 1.0.0)
Expand All @@ -129,19 +131,37 @@ GEM
coderay (1.1.3)
concurrent-ruby (1.2.3)
connection_pool (2.4.1)
crack (0.4.5)
rexml
cube-ruby (0.0.3)
dante (0.2.0)
date (3.3.4)
docile (1.4.0)
domain_name (0.6.20240107)
ed25519 (1.3.0)
faraday (2.8.1)
base64
faraday-net_http (>= 2.0, < 3.1)
faraday (1.10.3)
faraday-em_http (~> 1.0)
faraday-em_synchrony (~> 1.0)
faraday-excon (~> 1.1)
faraday-httpclient (~> 1.0)
faraday-multipart (~> 1.0)
faraday-net_http (~> 1.0)
faraday-net_http_persistent (~> 1.0)
faraday-patron (~> 1.0)
faraday-rack (~> 1.0)
faraday-retry (~> 1.0)
ruby2_keywords (>= 0.0.4)
faraday-net_http (3.0.2)
faraday-retry (2.2.0)
faraday (~> 2.0)
faraday-em_http (1.0.0)
faraday-em_synchrony (1.0.0)
faraday-excon (1.1.0)
faraday-httpclient (1.0.1)
faraday-multipart (1.0.4)
multipart-post (~> 2)
faraday-net_http (1.0.1)
faraday-net_http_persistent (1.2.0)
faraday-patron (1.0.0)
faraday-rack (1.0.0)
faraday-retry (1.0.3)
ffi (1.16.3)
gapic-common (0.21.1)
faraday (>= 1.9, < 3.a)
Expand All @@ -165,15 +185,14 @@ GEM
google-cloud-env (2.1.1)
faraday (>= 1.0, < 3.a)
google-cloud-errors (1.4.0)
google-protobuf (3.25.3-aarch64-linux)
google-protobuf (3.25.3-arm64-darwin)
google-protobuf (3.25.3-x86_64-darwin)
google-protobuf (3.25.3-x86_64-linux)
googleapis-common-protos (1.5.0)
google-protobuf (~> 3.18)
googleapis-common-protos-types (~> 1.7)
grpc (~> 1.41)
googleapis-common-protos-types (1.13.0)
googleapis-common-protos-types (1.14.0)
google-protobuf (~> 3.18)
googleauth (1.11.0)
faraday (>= 1.0, < 3.a)
Expand All @@ -182,9 +201,6 @@ GEM
multi_json (~> 1.11)
os (>= 0.9, < 2.0)
signet (>= 0.16, < 2.a)
grpc (1.62.0-aarch64-linux)
google-protobuf (~> 3.25)
googleapis-common-protos-types (~> 1.0)
grpc (1.62.0-arm64-darwin)
google-protobuf (~> 3.25)
googleapis-common-protos-types (~> 1.0)
Expand All @@ -197,10 +213,11 @@ GEM
haml (5.2.2)
temple (>= 0.8.0)
tilt
hashdiff (1.1.0)
http-accept (1.7.0)
http-cookie (1.0.5)
domain_name (~> 0.5)
i18n (0.9.5)
i18n (1.14.4)
concurrent-ruby (~> 1.0)
json (2.7.1)
json-schema (2.8.1)
Expand All @@ -224,11 +241,14 @@ GEM
mime-types-data (~> 3.2015)
mime-types-data (3.2024.0305)
mini_mime (1.1.5)
minitest (4.7.5)
minitest (5.22.3)
minitest-hooks (1.5.1)
minitest (> 5.3)
minitest-stub_any_instance (1.0.3)
mlanett-redis-lock (0.2.7)
redis
multi_json (1.15.0)
multipart-post (2.4.0)
mutex_m (0.2.0)
net-http-persistent (2.9.4)
net-imap (0.4.10)
Expand All @@ -247,7 +267,8 @@ GEM
net-ssh (7.2.1)
netrc (0.11.0)
newrelic_rpm (9.7.1)
oj (3.16.1)
oj (3.16.3)
bigdecimal (>= 3.0)
omni_logger (0.1.4)
logger
os (1.1.4)
Expand Down Expand Up @@ -357,10 +378,11 @@ GEM
net-ssh (>= 2.8.0)
systemu (2.6.5)
temple (0.10.3)
thread_safe (0.3.6)
tilt (2.3.0)
timeout (0.4.1)
tzinfo (2.0.6)
concurrent-ruby (~> 1.0)
tzinfo (1.2.11)
thread_safe (~> 0.1)
unicode-display_width (2.5.0)
unicorn (6.1.0)
kgio (~> 2.6)
Expand All @@ -370,38 +392,43 @@ GEM
unicorn (>= 4, < 7)
uuid (2.3.9)
macaddr (~> 1.0)
webmock (3.23.0)
addressable (>= 2.8.0)
crack (>= 0.3.2)
hashdiff (>= 0.4.0, < 2.0.0)

PLATFORMS
aarch64-linux
arm64-darwin-22
arm64-darwin-23
x86_64-darwin-18
x86_64-darwin-21
x86_64-darwin-23
x86_64-linux

DEPENDENCIES
activesupport (~> 3.2)
activesupport (~> 5)
bcrypt_pbkdf (>= 1.0, < 2.0)
bigdecimal (= 1.4.2)
bigdecimal
capistrano (~> 3)
capistrano-bundler
capistrano-locally
capistrano-rbenv
crack (= 0.4.5)
cube-ruby
ed25519 (>= 1.2, < 2.0)
faraday (~> 1.9)
ffi
goo!
haml (~> 5.2.2)
json-schema (~> 2.0)
minitest (~> 4.0)
minitest (~> 5.0)
minitest-hooks (~> 1.5)
minitest-stub_any_instance
multi_json (~> 1.0)
multi_json
ncbo_annotator!
ncbo_cron!
ncbo_ontology_recommender!
newrelic_rpm
oj (~> 3.0)
oj
ontologies_linked_data!
parseconfig
rack
Expand All @@ -428,6 +455,7 @@ DEPENDENCIES
sparql-client!
unicorn
unicorn-worker-killer
webmock

BUNDLED WITH
2.4.22
4 changes: 2 additions & 2 deletions test/controllers/test_annotator_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

class TestAnnotatorController < TestCase

def self.before_suite
def before_suite
@@redis = Redis.new(:host => Annotator.settings.annotator_redis_host, :port => Annotator.settings.annotator_redis_port)
db_size = @@redis.dbsize
if db_size > MAX_TEST_REDIS_SIZE
Expand All @@ -20,7 +20,7 @@ def self.before_suite
annotator = Annotator::Models::NcboAnnotator.new
annotator.init_redis_for_tests()
annotator.create_term_cache_from_ontologies(@@ontologies, false)
mapping_test_set
self.class.mapping_test_set
end

def test_annotate
Expand Down
2 changes: 1 addition & 1 deletion test/controllers/test_batch_controller.rb
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
require_relative '../test_case'

class TestBatchController < TestCase
def self.before_suite
def before_suite
LinkedData::SampleData::Ontology.delete_ontologies_and_submissions
@@ontologies = LinkedData::SampleData::Ontology.sample_owl_ontologies
end
Expand Down
2 changes: 1 addition & 1 deletion test/controllers/test_classes_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

class TestClassesController < TestCase

def self.before_suite
def before_suite
options = {ont_count: 1,
submission_count: 3,
submissions_to_process: [1, 2],
Expand Down
2 changes: 1 addition & 1 deletion test/controllers/test_instances_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

class TestInstancesController < TestCase

def self.before_suite
def before_suite
LinkedData::SampleData::Ontology.create_ontologies_and_submissions({
process_submission: true,
acronym: 'XCT-TEST-INST',
Expand Down
4 changes: 2 additions & 2 deletions test/controllers/test_mappings_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@

class TestMappingsController < TestCase

def self.before_suite

def before_suite
self.backend_4s_delete
["BRO-TEST-MAP-0","CNO-TEST-MAP-0","FAKE-TEST-MAP-0"].each do |acr|
LinkedData::Models::OntologySubmission.where(ontology: [acronym: acr]).to_a.each do |s|
s.delete
Expand Down
2 changes: 1 addition & 1 deletion test/controllers/test_metrics_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

class TestMetricsController < TestCase

def self.before_suite
def before_suite
if OntologySubmission.all.count > 100
puts "this test is going to wipe out all submission and ontologies. probably this is not a test env."
return
Expand Down
6 changes: 3 additions & 3 deletions test/controllers/test_notes_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@

class TestNotesController < TestCase

def self.before_suite
self.new("before_suite").delete_ontologies_and_submissions
@@ontology, @@cls = self.new("before_suite")._ontology_and_class
def before_suite
self.delete_ontologies_and_submissions
@@ontology, @@cls = self._ontology_and_class

@@note_user = "test_note_user"
@@user = LinkedData::Models::User.new(
Expand Down
10 changes: 5 additions & 5 deletions test/controllers/test_ontologies_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,11 +2,11 @@
require_relative '../test_case'

class TestOntologiesController < TestCase
def self.before_suite
_set_vars
_delete
_create_user
_create_onts
def before_suite
self.class._set_vars
self.class._delete
self.class._create_user
self.class._create_onts
end

def teardown
Expand Down
8 changes: 4 additions & 4 deletions test/controllers/test_ontology_analytics_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -196,7 +196,7 @@ class TestOntologyAnalyticsController < TestCase
}
}

def self.before_suite
def before_suite
@@redis = Redis.new(:host => Annotator.settings.annotator_redis_host, :port => Annotator.settings.annotator_redis_port)
db_size = @@redis.dbsize
if db_size > MAX_TEST_REDIS_SIZE
Expand All @@ -212,9 +212,9 @@ def self.before_suite
"SNOMEDCT" => "SNOMEDCT Ontology",
"TST" => "TST Ontology"
}
_delete
_create_user
_create_onts
self.class._delete
self.class._create_user
self.class._create_onts
end

def teardown
Expand Down
9 changes: 5 additions & 4 deletions test/controllers/test_ontology_submissions_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,11 @@

class TestOntologySubmissionsController < TestCase

def self.before_suite
_set_vars
_create_user
_create_onts
def before_suite
self.backend_4s_delete
self.class._set_vars
self.class._create_user
self.class._create_onts
end

def self._set_vars
Expand Down
4 changes: 2 additions & 2 deletions test/controllers/test_properties_controller.rb
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@

class TestPropertiesController < TestCase

def self.before_suite
def before_suite
count, acronyms, bro = LinkedData::SampleData::Ontology.create_ontologies_and_submissions({
process_submission: true,
acronym: "BROSEARCHTEST",
Expand All @@ -25,7 +25,7 @@ def self.before_suite
@@acronyms = @@ontologies.map { |ont| ont.bring_remaining; ont.acronym }
end

def self.after_suite
def after_suite
LinkedData::SampleData::Ontology.delete_ontologies_and_submissions
end

Expand Down
Loading
Loading