[models] update FontNameModel flags method to address qabstractmodelitemtester fail

Author: chrissimpkins

src/slice/models.py

@@ -146,8 +146,14 @@ def headerData(self, section, orientation, role):
             return "Instance Values"
 
     def flags(self, index):
+        # Note: index validity checks in this block address qabstractitemmodeltester error:
+        # QtWarningMsg: FAIL! flags == Qt::ItemIsDropEnabled || flags == 0 () returned FALSE (qabstractitemmodeltester.cpp:329)
+
         # all indices are editable in this table
-        return super().flags(index) | Qt.ItemIsEditable
+        if index.isValid():
+            return super().flags(index) | Qt.ItemIsEditable
+        else:
+            return super().flags(index)
 
     def get_version(self):
         return self.font_version.split(";")[0]

tests/test_models_fontname.py

@@ -0,0 +1,18 @@
+from pathlib import Path
+
+from PyQt5.QtWidgets import QTableView
+
+from slice.models import FontNameModel, FontModel
+
+
+def get_font_model():
+    return FontModel(Path("tests/assets/fonts/Recursive-VF.subset.ttf").resolve())
+
+
+def test_fontname_model_default(qtbot, qtmodeltester):
+    tableview = QTableView()
+    model = FontNameModel()
+    tableview.setModel(model)
+    qtbot.addWidget(tableview)
+    # check model with default instantiation
+    qtmodeltester.check(model)