Skip to content

Commit

Permalink
✅ Updating tests
Browse files Browse the repository at this point in the history
  • Loading branch information
JohnGiorgi committed Aug 15, 2019
1 parent 95e4385 commit 9228ca0
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 8 deletions.
8 changes: 4 additions & 4 deletions saber/tests/test_bert_for_ner.py
Original file line number Diff line number Diff line change
Expand Up @@ -321,16 +321,16 @@ def test_predict_mt(self, bert_for_ner_specify_mt):

def test_prepare_optimizers(self, bert_for_ner_specify):
"""Asserts that the returned optimizer object is as expected after call to
`BertForNER.prepare_optimizers()`.
`BertForNER._prepare_optimizers()`.
"""
actual = bert_for_ner_specify.prepare_optimizers()
actual = bert_for_ner_specify._prepare_optimizers()

assert all(isinstance(opt, AdamW) for opt in actual)

def test_prepare_optimizers_mt(self, bert_for_ner_specify_mt):
"""Asserts that the returned optimizer object is as expected after call to
`BertForNER.prepare_optimizers()` for a multi-task model.
`BertForNER._prepare_optimizers()` for a multi-task model.
"""
actual = bert_for_ner_specify_mt.prepare_optimizers()
actual = bert_for_ner_specify_mt._prepare_optimizers()

assert all(isinstance(opt, AdamW) for opt in actual)
6 changes: 3 additions & 3 deletions saber/tests/test_bert_for_ner_and_re.py
Original file line number Diff line number Diff line change
Expand Up @@ -212,14 +212,14 @@ def test_predict_mt(self):

def test_prepare_optimizers(self, bert_for_ner_and_re_specify):
"""Asserts that the returned optimizer object is as expected after call to
`BertForNERAndRE.prepare_optimizers()`.
`BertForNERAndRE._prepare_optimizers()`.
"""
actual = bert_for_ner_and_re_specify.prepare_optimizers()
actual = bert_for_ner_and_re_specify._prepare_optimizers()

assert all(isinstance(opt, AdamW) for opt in actual)

def test_prepare_optimizers_mt(self):
"""Asserts that the returned optimizer object is as expected after call to
`BertForNERAndRE.prepare_optimizers()` for a multi-task model.
`BertForNERAndRE._prepare_optimizers()` for a multi-task model.
"""
pass
2 changes: 1 addition & 1 deletion saber/tests/test_bert_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ def test_get_dataloader_for_bert_conll2004(self, conll2004datasetreader_load, be

def test_get_bert_optimizer(self, bert_for_ner_specify):
"""Asserts that the returned optimizer object is as expected after call to
`BertForNER.prepare_optimizers()` for a single-task model.
`bert_utils.get_bert_optimizer()` for a single-task model.
"""
model, config = bert_for_ner_specify.model, bert_for_ner_specify.config

Expand Down

0 comments on commit 9228ca0

Please sign in to comment.