diff options
| author | Frederik Gladhorn <frederik.gladhorn@qt.io> | 2019-10-15 09:56:58 +0200 |
|---|---|---|
| committer | Frederik Gladhorn <frederik.gladhorn@qt.io> | 2019-10-16 10:11:42 +0200 |
| commit | 4ce6aaf3bc589db5527c31c69553bc51bab08c46 (patch) | |
| tree | b74ba75eeb3dad501fa3486670c4d60255cc3c4d /src | |
| parent | 3f42e54db5e4780e37381e00598a69d93dfc9c0e (diff) | |
Handle err in all instances
While it's not important in the test, it makes sense to make sure all
variables are used everywhere to make staticcheck happy. In the Coin
code we had issues with shadowed error variables, so now I'm paranoid.
Change-Id: Ia5d32375b7659e048fbcd3fbebcbd6323c16123c
Reviewed-by: Simon Hausmann <simon.hausmann@qt.io>
Diffstat (limited to 'src')
| -rw-r--r-- | src/qtmoduleupdater/module_test.go | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/qtmoduleupdater/module_test.go b/src/qtmoduleupdater/module_test.go index 31849479..0e665642 100644 --- a/src/qtmoduleupdater/module_test.go +++ b/src/qtmoduleupdater/module_test.go @@ -77,6 +77,7 @@ func TestProposedUpdateFailsForModulesThatDependOnMoreThanQtBase(t *testing.T) { qtSvg, ok := todoMap["qt/qtsvg"] assert.True(t, ok, "qtsvg must be present in the module map") yamlObject, err := qtSvg.maybePrepareUpdatedDependenciesYaml(availableModules) + assert.Nil(t, err, "No error expected preparing dependencies.yaml update") assert.NotNil(t, yamlObject, "Yaml object must be defined for qtsvg") yamlStr, err := yamlObject.ToString() |