Project

General

Profile

Bug #1501

d1_integration hudson failure wrt object format

Added by Rob Nahf about 13 years ago. Updated almost 13 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Chris Jones
Category:
-
Target version:
Sprint-2011.18-Block.3
Start date:
Due date:
% Done:

100%

Milestone:
Product Version:
*
Story Points:
Sprint:

Description

been failing Hudson for a while:

org.dataone.integration.D1ClientTest.testCreateDescribedDataAndMetadata

assertion failed for host http://cn-dev.dataone.org/knb/d1/ Expected: is "text/csv" got: "text/plain"

The question is did the object format change and the new result is what we should expect, or is something wrong?

History

#1 Updated by Rob Nahf about 13 years ago

  • Assignee set to Chris Jones

#2 Updated by Dave Vieglais about 13 years ago

  • Target version changed from Sprint-2011.14-Block.2 to Sprint-2011.15-Block.2
  • Position set to 15
  • Position deleted (1)

#3 Updated by Dave Vieglais about 13 years ago

  • Position set to 1
  • Target version changed from Sprint-2011.15-Block.2 to Sprint-2011.17-Block.3
  • Position deleted (29)

#4 Updated by Chris Jones almost 13 years ago

  • Status changed from New to In Progress

#5 Updated by Chris Jones almost 13 years ago

MetacatHandler.createSystemMetadata() now parses EML documents to ferret out system metadata information such as the object format. The parsing code considers any simple-delimited text file to be of object format 'text/csv' (according to Matt's definition of CSV based on defacto use in ecology). Therefore, the test is expecting 'text/plain' when the parser is returning 'text/csv'.

We need to either:
1) Change the parser to consider tab or space delimited text files to be 'text/plain'

or

2) Change the test to expect 'text/csv' as the object format, accepting the definition that Matt described

#6 Updated by Dave Vieglais almost 13 years ago

  • Target version changed from Sprint-2011.17-Block.3 to Sprint-2011.18-Block.3
  • Position set to 183
  • Position deleted (5)

#7 Updated by Chris Jones almost 13 years ago

We reluctantly decided to adopt the second approach - use text/csv as a catch-all format for simple delimited text files since that is the norm for many researchers and scientific analysis packages (R, Matlab, SAS, etc.).

#8 Updated by Chris Jones almost 13 years ago

  • Status changed from In Progress to Closed

Also available in: Atom PDF

Add picture from clipboard (Maximum size: 14.8 MB)