Fix: RemoveScheme function in irmaclient already stripping storage be…

…fore checking whether scheme is in assets

irmaclient/client.go

@@ -1547,7 +1547,15 @@ func (client *Client) RemoveScheme(schemeID irma.SchemeManagerIdentifier) error
 		return errors.New("unknown scheme manager")
 	}
 
-	err := client.stripStorage([]irma.SchemeManagerIdentifier{schemeID}, true)
+	isInAssets, err := client.Configuration.IsInAssets(scheme)
+	if err != nil {
+		return err
+	}
+	if isInAssets {
+		return errors.New("cannot remove scheme manager that is in assets")
+	}
+
+	err = client.stripStorage([]irma.SchemeManagerIdentifier{schemeID}, true)
 	if err != nil {
 		return err
 	}

schemes.go

@@ -258,10 +258,18 @@ func (conf *Configuration) UpdateScheme(scheme Scheme, downloaded *IrmaIdentifie
 	return nil
 }
 
+func (conf *Configuration) IsInAssets(scheme Scheme) (bool, error) {
+	if conf.assets == "" {
+		return false, nil
+	}
+	_, exists, err := common.Stat(path.Join(conf.assets, scheme.id()))
+	return exists, err
+}
+
 // DangerousDeleteScheme deletes the given scheme from the configuration.
 // Be aware: this action is dangerous when the scheme is still in use.
 func (conf *Configuration) DangerousDeleteScheme(scheme Scheme) error {
-	_, exists, err := common.Stat(path.Join(conf.assets, scheme.id()))
+	exists, err := conf.IsInAssets(scheme)
 	if err != nil {
 		return err
 	}