Check file system before resizing: necessary for testResizePartition:shrink to pass

src/modules/partition/CMakeLists.txt

@@ -23,6 +23,7 @@ calamares_add_plugin( partition
     CONFIG_FILE module.conf
     SOURCES
         BootLoaderModel.cpp
+        CheckFileSystemJob.cpp
         CreatePartitionDialog.cpp
         CreatePartitionJob.cpp
         CreatePartitionTableJob.cpp

src/modules/partition/CheckFileSystemJob.cpp

@@ -0,0 +1,56 @@
+/* === This file is part of Calamares - <http://github.com/calamares> ===
+ *
+ *   Copyright 2014, Aurélien Gâteau <agateau@kde.org>
+ *
+ *   Calamares is free software: you can redistribute it and/or modify
+ *   it under the terms of the GNU General Public License as published by
+ *   the Free Software Foundation, either version 3 of the License, or
+ *   (at your option) any later version.
+ *
+ *   Calamares is distributed in the hope that it will be useful,
+ *   but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ *   GNU General Public License for more details.
+ *
+ *   You should have received a copy of the GNU General Public License
+ *   along with Calamares. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#include <CheckFileSystemJob.h>
+
+// CalaPM
+#include <core/partition.h>
+#include <fs/filesystem.h>
+#include <util/report.h>
+
+CheckFileSystemJob::CheckFileSystemJob( Partition* partition )
+    : PartitionJob( partition )
+{}
+
+QString
+CheckFileSystemJob::prettyName() const
+{
+    QString path = partition()->partitionPath();
+    return tr( "Checking file system on partition %1." ).arg( path );
+}
+
+Calamares::JobResult
+CheckFileSystemJob::exec()
+{
+    FileSystem& fs = partition()->fileSystem();
+
+    // if we cannot check, assume everything is fine
+    if ( fs.supportCheck() != FileSystem::cmdSupportFileSystem )
+        return Calamares::JobResult::ok();
+
+    Report report( nullptr );
+    bool ok = fs.check( report, partition()->partitionPath() );
+    if ( !ok )
+        return Calamares::JobResult::error(
+                   tr( "The file system check on partition %1 failed." )
+                   .arg( partition()->partitionPath() ),
+                   report.toText()
+               );
+
+    return Calamares::JobResult::ok();
+}

src/modules/partition/CheckFileSystemJob.h

@@ -0,0 +1,33 @@
+/* === This file is part of Calamares - <http://github.com/calamares> ===
+ *
+ *   Copyright 2014, Aurélien Gâteau <agateau@kde.org>
+ *
+ *   Calamares is free software: you can redistribute it and/or modify
+ *   it under the terms of the GNU General Public License as published by
+ *   the Free Software Foundation, either version 3 of the License, or
+ *   (at your option) any later version.
+ *
+ *   Calamares is distributed in the hope that it will be useful,
+ *   but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *   MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ *   GNU General Public License for more details.
+ *
+ *   You should have received a copy of the GNU General Public License
+ *   along with Calamares. If not, see <http://www.gnu.org/licenses/>.
+ */
+
+#ifndef CHECKFILESYSTEMJOB_H
+#define CHECKFILESYSTEMJOB_H
+
+#include <PartitionJob.h>
+
+class CheckFileSystemJob : public PartitionJob
+{
+public:
+    CheckFileSystemJob( Partition* partition );
+
+    QString prettyName() const override;
+    Calamares::JobResult exec() override;
+};
+
+#endif /* CHECKFILESYSTEMJOB_H */

src/modules/partition/ResizePartitionJob.cpp

@@ -40,6 +40,7 @@
 
 #include <ResizePartitionJob.h>
 
+#include <CheckFileSystemJob.h>
 #include <MoveFileSystemJob.h>
 #include <utils/Logger.h>
 
@@ -229,6 +230,7 @@ ResizePartitionJob::exec()
 
     // Create jobs
     QList< Calamares::job_ptr > jobs;
+    jobs << Calamares::job_ptr( new CheckFileSystemJob( partition() ) );
     if ( m_partition->roles().has( PartitionRole::Extended ) )
         jobs << Calamares::job_ptr( new SetPartGeometryJob( &context, m_newFirstSector, newLength ) );
     else

src/modules/partition/tests/CMakeLists.txt

@@ -2,6 +2,7 @@ find_package( Qt5 COMPONENTS Test REQUIRED )
 include( ECMAddTests )
 
 set( jobtests_SRCS
+    ${PartitionModule_SOURCE_DIR}/CheckFileSystemJob.cpp
     ${PartitionModule_SOURCE_DIR}/CreatePartitionJob.cpp
     ${PartitionModule_SOURCE_DIR}/CreatePartitionTableJob.cpp
     ${PartitionModule_SOURCE_DIR}/DeletePartitionJob.cpp