Add a test for Odds_XML-long-import.xml and remove it from the TODO.

[dead/htsn-import.git] / doc / TODO

diff --git a/doc/TODO b/doc/TODO
index 817367d29643e1a0d398de19077d24a74f5e2c90..6748d67eda3c889b741e661f556052fde24dbb53 100644 (file)
--- a/doc/TODO
+++ b/doc/TODO
@@ -1,33 +1,12 @@
-1. Re-enable DTD validation once
-   https://github.com/UweSchmidt/hxt/issues/15 is fixed.
-
-2. Write a test for test/xml/Odds_XML-long-import.xml once it no
-   longer takes 10 minutes to import (Postgres only?).
-
-3. We have DTDs but no sample XML for the following SportInfo types,
+1. We have DTDs but no sample XML for the following SportInfo types,
    which have therefore been left unimplmented for now:
 
-   * Cbask_Indv_No_Avg_XML.dtd
-   * Cbask_Indv_Shooting_XML.dtd
    * CBASK_StealsXML.dtd
-   * Cbask_Team_Scoring_Rebound_Margin_XML.dtd
-   * Cbask_Team_Scoring_XML.dtd
-   * Cbask_Team_Shooting_Pct_XML.dtd
-   * NFLInterceptionLeadersXML.dtd
-   * NFLPassingLeadersXML.dtd
-   * NFLReceivingLeadersXML.dtd
-   * NFLRushingLeadersXML.dtd
-   * NFLTackleFFLeadersXML.dtd
-   * NFLTopKickoffReturnXML.dtd
-   * NFLTopPuntReturnXML.dtd
-   * NFLYardsXML.dtd
-   * NFL_PuntingLeaders_XML.dtd
 
-4. The following DTD types were handled (in some form) by the old
+2. The following DTD types were handled (in some form) by the old
    FeedGrabber. They are not yet handled by htsn-import (some may not
    be valid):
 
-   * AutoRacingDriverList
    * AutoRacingGridXML
    * CBASK_Individual_Stats_XML
    * Cbask_Roster_XML
@@ -60,10 +39,13 @@
    * WNBA_Individual_Stats_XML
    * WNBATeamScheduleXML
 
-5. Consolidate all of the make_game_time functions which take a
+3. Consolidate all of the make_game_time functions which take a
    date/time and produce a combined time.
 
-6. Factor out test code where possible; a lot of them differ only in
+4. Factor out test code where possible; a lot of them differ only in
    the filename.
 
-7. Combine test XML files where possible.
+5. Combine test XML files where possible. We don't need to e.g. import
+   two copies of newsxml just because one has an empty attribute that
+   we want to test: we could just delete an attribute from the first
+   file.