Project

General

Profile

Actions

task #578

open

@Ignore removed from PersistentTermInitializerTest

Added by Andreas Müller about 15 years ago. Updated over 2 years ago.

Status:
In Progress
Priority:
Priority14
Category:
cdmlib
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Severity:
normal

Description

Hi Ben,

Yes probably you right. I didn't really go into the PersistentTermInitializerTest problem but tried it within eclipse with the new override.properties and it worked whereas in maven it doesn't

Caused by: java.lang.ClassCastException: org.hibernate.collection.PersistentSet cannot be cast to java.util.SortedSet

  at eu.etaxonomy.cdm.model.common.OrderedTermVocabulary.addTerm(OrderedTermVocabulary.java:154)


  at eu.etaxonomy.cdm.model.common.OrderedTermVocabulary.addTerm(OrderedTermVocabulary.java:1)


  at eu.etaxonomy.cdm.database.PersistentTermInitializer.firstPass(PersistentTermInitializer.java:141)


  at eu.etaxonomy.cdm.database.PersistentTermInitializer.initialize(PersistentTermInitializer.java:87)

I will leave the test as @Ignored for now.

Cheers,

Andreas

----Ursprüngliche Nachricht-----

Von: Ben Clark B.Clark@kew.org

Gesendet: Donnerstag, 5. Februar 2009 11:07

An: 'dev-cdmlib@mnhn.fr'

Betreff: RE: [dev-cdmlib] I want to commit some major changes to cdmlib term loading, please could you confirm that this is ok?

Hi Andreas,

For PersistentTermInitializerTest I had to create a new override.properties because for this test it looks like > we need term initializing. Or do I think the wrong way here?

I'm not sure what to do about PersistentTermInitializerTest, really, as running it side-by-side with the other tests (which use testing term initializer) results in some issues. I think disabling it using override.properties simply checks against the terms initialized by TestingTermInitializer, not actually testing the PersistentTermInitializer at all.

I just @Ignore -ed it, not a perfect solution, but the whole term initialization scenario is quite complex and I couldn't think of a better idea. Running the other tests properly is more of a priority, I think.

Cheers,

   Ben
Actions #1

Updated by Andreas Müller over 14 years ago

  • Status changed from New to In Progress
  • Priority changed from Priority11 to Priority09
Actions #2

Updated by Andreas Müller over 14 years ago

  • Target version changed from CDM lib Release 2.0 to CDM lib Release 2.3
Actions #3

Updated by Andreas Müller over 2 years ago

  • Description updated (diff)
  • Priority changed from Priority09 to Priority14

The test is still on ignore.

Actions #4

Updated by Andreas Müller over 2 years ago

  • Target version changed from cdmlib - Old Next Major Release to Unassigned CDM tickets
Actions #5

Updated by Andreas Müller over 2 years ago

  • Private changed from Yes to No
Actions

Also available in: Atom PDF