Changeset 54

Timestamp:

04/14/06 05:08:13 (2 years ago)

Author:

jordi

Message:

Increment ePDFView's version number.
The MainPterTest? doesn't include the epdfview.h header file, because I thought that it was the problem about the memory used by the compiler. It turns out that the problem was that some functions are TOO LONG and the compiler maxs out the memory (128MiB, thats it) and the system starts to swapping. Now I have reduced the compile time from 3 minutes and halt to 1 minute, and the system doesn't freeze :-)

Location:

trunk

Files:

• configure.ac (modified) (1 diff)
• src/IDocument.h (modified) (1 diff)
• tests/MainPterTest.cxx (modified) (12 diffs)
• tests/MainPterTest.h (modified) (2 diffs)

trunk/configure.ac

@@ -1 +1 @@
 dnl Process this file with autoconf to produce a configure script.
-AC_INIT([ePDFView], [0.1.0], [jordi@emma-soft.com])
+AC_INIT([ePDFView], [0.1.1], [jordi@emma-soft.com])
 AC_PREREQ([2.13])
 AC_CONFIG_HEADER([config.h])

trunk/src/IDocument.h

@@ -24 +24 @@
 namespace ePDFView 
 {
+    // Forward declarations.
+    class DocumentPage;
+    class DocumentIndex;
+
     ///
     /// @brief Defines the possible errors loading a document.

trunk/tests/MainPterTest.cxx

@@ -16 +16 @@
 // Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
 
-#include <epdfview.h>
+#include <glib.h>
+#include <IDocument.h>
+#include <IMainView.h>
+#include <MainPter.h>
 #include "DumbDocument.h"
 #include "DumbMainView.h"
@@ -46 +49 @@
     // We only need to delete the presenter, as all other
     // classes are deleted by it.
-//    delete m_MainPter;
-}
+    delete m_MainPter;
+}
+
 
 ///
@@ -295 +299 @@
     CPPUNIT_ASSERT_EQUAL (1, m_View->getCurrentPage ());
     CPPUNIT_ASSERT (m_View->hasImagePageView ());
+    // This is to check that after calling this function, it returns false
+    // because no page can be shown between the two... (I didn't called
+    // the presenter or the view, yet)
     CPPUNIT_ASSERT (!m_View->hasImagePageView ());
     // Going to the next page should make all actions sensitive.
@@ -305 +312 @@
     CPPUNIT_ASSERT (m_View->isSensitiveGoToPreviousPage ());
     CPPUNIT_ASSERT (m_View->hasImagePageView ());
-    CPPUNIT_ASSERT (!m_View->hasImagePageView ());
     // Going to the last will make some unsensitive.
     m_MainPter->goToLastPageActivated ();
@@ -315 +321 @@
     CPPUNIT_ASSERT (m_View->isSensitiveGoToPreviousPage ());
     CPPUNIT_ASSERT (m_View->hasImagePageView ());
-    CPPUNIT_ASSERT (!m_View->hasImagePageView ());
     // Again to the last won't harm (just in case...) nor going next.
     m_MainPter->goToLastPageActivated ();
@@ -358 +363 @@
     CPPUNIT_ASSERT (m_View->hasImagePageView ());
     CPPUNIT_ASSERT (!m_View->hasImagePageView ());
+}
+
+///
+/// @brief Test the page navigation using the entry on the toolbar.
+///
+/// The application has a text entry that the user can use to to a 
+/// page just by writting the number. Any non number after the first number
+/// character will be discarded.
+///
+void
+MainPterTest::pageNavigationEntry ()
+{
+    m_View->setOpenFileName ("/tmp/test.pdf");
+    m_Document->setNumPages (4);
+    m_MainPter->openFileActivated ();
+    // Check that sets the correct number of pages and the current page.
+    CPPUNIT_ASSERT_EQUAL (4, m_View->getTotalPages ());
+    CPPUNIT_ASSERT_EQUAL (1, m_View->getCurrentPage ());
+    CPPUNIT_ASSERT (m_View->hasImagePageView ());
 
     // Now try again using the page entry on toolbar.
-    m_View->setGoToPageText ("2");
+    m_View->setGoToPageText ("2 of 4");
     m_MainPter->goToPageActivated ();
     CPPUNIT_ASSERT_EQUAL (2, m_View->getCurrentPage ()); 
@@ -371 +395 @@
     CPPUNIT_ASSERT (!m_View->hasImagePageView ());
 
-    m_View->setGoToPageText ("4");
+    m_View->setGoToPageText ("4 of 4");
     m_MainPter->goToPageActivated ();
     CPPUNIT_ASSERT_EQUAL (4, m_View->getCurrentPage ()); 
@@ -382 +406 @@
     CPPUNIT_ASSERT (!m_View->hasImagePageView ());
 
-    m_View->setGoToPageText ("1");
+    m_View->setGoToPageText ("1 of 4");
     m_MainPter->goToPageActivated ();
     CPPUNIT_ASSERT_EQUAL (1, m_View->getCurrentPage ()); 
@@ -394 +418 @@
 
     // Invalid values.
-    m_View->setGoToPageText ("123123");
+    m_View->setGoToPageText ("123123 of 1");
     m_MainPter->goToPageActivated ();
     CPPUNIT_ASSERT_EQUAL (4, m_View->getCurrentPage ()); 
@@ -405 +429 @@
     CPPUNIT_ASSERT (!m_View->hasImagePageView ());
 
-    m_View->setGoToPageText ("0");
+    m_View->setGoToPageText ("0 of 12");
     m_MainPter->goToPageActivated ();
     CPPUNIT_ASSERT_EQUAL (1, m_View->getCurrentPage ()); 
@@ -417 +441 @@
 
     m_MainPter->goToNextPageActivated ();
-    m_View->setGoToPageText ("Jejej");
+    m_View->setGoToPageText ("Jejej :-)");
     m_MainPter->goToPageActivated ();
     CPPUNIT_ASSERT_EQUAL (1, m_View->getCurrentPage ()); 
@@ -527 +551 @@
     CPPUNIT_ASSERT (!m_View->hasImagePageView ());
 }
+

trunk/tests/MainPterTest.h

@@ -34 +34 @@
         CPPUNIT_TEST (goodPassword);
         CPPUNIT_TEST (pageNavigation);
+        CPPUNIT_TEST (pageNavigationEntry);
         CPPUNIT_TEST (pageRotate);
         CPPUNIT_TEST (pageZoom);
@@ -50 +51 @@
             void goodPassword (void);
             void pageNavigation (void);
+            void pageNavigationEntry (void);
             void pageRotate (void);
             void pageZoom (void);