[lmi-commits] [6112] Remove old code to make way for new

[Greg Chicares]

Revision: 6112
          http://svn.sv.gnu.org/viewvc/?view=rev&root=lmi&revision=6112
Author:   chicares
Date:     2015-03-01 17:29:17 +0000 (Sun, 01 Mar 2015)
Log Message:
-----------
Remove old code to make way for new

Modified Paths:
--------------
    lmi/trunk/ChangeLog
    lmi/trunk/wx_test_validate_output.cpp

Modified: lmi/trunk/ChangeLog
===================================================================
--- lmi/trunk/ChangeLog 2015-03-01 16:33:35 UTC (rev 6111)
+++ lmi/trunk/ChangeLog 2015-03-01 17:29:17 UTC (rev 6112)
@@ -35682,3 +35682,9 @@
 Remove platform-specific code that has become unnecessary. See:
   http://lists.nongnu.org/archive/html/lmi/2015-02/msg00009.html
 
+20150301T1729Z <address@hidden> [516]
+
+  wx_test_validate_output.cpp
+Remove old code to make way for new. See:
+  http://lists.nongnu.org/archive/html/lmi/2015-02/msg00012.html
+

Modified: lmi/trunk/wx_test_validate_output.cpp
===================================================================
--- lmi/trunk/wx_test_validate_output.cpp       2015-03-01 16:33:35 UTC (rev 
6111)
+++ lmi/trunk/wx_test_validate_output.cpp       2015-03-01 17:29:17 UTC (rev 
6112)
@@ -37,44 +37,6 @@
 #include <wx/testing.h>
 #include <wx/uiaction.h>
 
-/*
-    This implements the following item of the testing specification:
-
-        14. Validate existence and naming conventions of output for
-            named and unnamed files.
-
-          A. File | New | Illustration | enter 'idiosyncrasyZ' in
-             'Comments' | OK
-             Expected results:
-               file 'unnamed.monthly_trace.tsv' exists
-
-OMIT B: duplicates A above:
-OMIT      B. File | Open | 'MonthlyTrace.ill' | press 'OK' to
-OMIT          [dismiss message box] | press 'OK' to run illustration
-OMIT         Expected results:
-OMIT           file 'MonthlyTrace.monthly_trace.tsv' exists
-
-          C. File | New | MEC testing | OK
-             Expected results:
-               file 'unnamed.mec.tsv' exists
-
-OMIT D: duplicates C above:
-OMIT      D. File | Open | 'MecTesting.mec' | OK
-OMIT         Expected results:
-OMIT           file 'MecTesting.mec.tsv' exists
-
-    The currently configured spreadsheet file extension is used instead of the
-    hard-coded "tsv", otherwise the tests are implemented exactly as specified.
- */
-
-// ERASE THIS BLOCK COMMENT WHEN IMPLEMENTATION COMPLETE. The block
-// comment below changes the original specification, and does not
-// yet describe the present code. Desired changes:
-//  - Remove code corresponding to B and D above.
-//  - Then remove the entire /*...*/ comment block above (its
-//    remaining contents are incorporated in the first two
-//    paragraphs below).
-//  - Code the other tests specified below.
 // Consider renaming this file to 'wx_test_spreadsheet_output.cpp'
 // e.g., because its purpose is to test *spreadsheet* output only.
 // To us at least, to "validate" a file suggests checking its
@@ -101,6 +63,8 @@
 /// Verify that this file was created:
 ///   unnamed.mec.tsv
 ///
+/// THE TESTS BELOW ARE NOT IMPLEMENTED YET!
+///
 /// File | New | Census
 /// Census | Edit case defaults
 ///   Comments: "idiosyncrasyZ"
@@ -216,32 +180,6 @@
 
     // And check that this resulted in the creation of the expected file.
     LMI_ASSERT(unnamed_trace.exists());
-
-    // Open an existing illustration already containing the same comment.
-    wxUIActionSimulator ui;
-
-    output_file_existence_checker
-        existing_trace("MonthlyTrace.monthly_trace" + ext);
-
-    ui.Char('o', wxMOD_CONTROL);    // "File|Open"
-    wxTEST_DIALOG
-        (wxYield()
-        
,wxExpectModal<wxFileDialog>(get_test_file_path_for("MonthlyTrace.ill")).
-            Describe("illustration open file dialog")
-        ,wxExpectModal<wxMessageDialog>(wxID_OK).
-            Describe("warning message box after opening the illustration")
-        ,wxExpectDismissableModal<MvcController>(wxID_OK).
-            Describe("illustration parameters dialog")
-        );
-
-    ui.Char('l', wxMOD_CONTROL);    // "File|Close"
-    wxTEST_DIALOG
-        (wxYield()
-        ,wxExpectModal<wxMessageDialog>(wxNO).
-            Describe("message box confirming closing modified illustration")
-        );
-
-    LMI_ASSERT(existing_trace.exists());
 }
 
 /// Validate spreadsheet output for MEC testing.
@@ -276,21 +214,4 @@
     wxYield();
 
     LMI_ASSERT(unnamed_output.exists());
-
-    // And when opening an existing one.
-    output_file_existence_checker existing_output("MecTesting.mec" + ext);
-
-    ui.Char('o', wxMOD_CONTROL);    // "File|Open"
-    wxTEST_DIALOG
-        (wxYield()
-         
,wxExpectModal<wxFileDialog>(get_test_file_path_for("MecTesting.mec")).
-            Describe("MEC open file dialog")
-         ,wxExpectDismissableModal<MvcController>(wxID_OK).
-            Describe("MEC parameters dialog")
-        );
-
-    ui.Char('l', wxMOD_CONTROL);    // "File|Close"
-    wxYield();
-
-    LMI_ASSERT(existing_output.exists());
 }