moderize error handling with errors.As/Is

README.md

@@ -29,7 +29,7 @@ item.SetSynchronizable(keychain.SynchronizableNo)
 item.SetAccessible(keychain.AccessibleWhenUnlocked)
 err := keychain.AddItem(item)
 
-if err == keychain.ErrorDuplicateItem {
+if errors.Is(err, keychain.ErrorDuplicateItem) {
   // Duplicate
 }
 ```
@@ -98,7 +98,7 @@ item := keychain.NewGenericPassword("MyService", "gabriel", "A label", []byte("t
 item.SetSynchronizable(keychain.SynchronizableNo)
 item.SetAccessible(keychain.AccessibleWhenUnlocked)
 err := keychain.AddItem(item)
-if err == keychain.ErrorDuplicateItem {
+if errors.Is(err, keychain.ErrorDuplicateItem) {
   // Duplicate
 }
 
@@ -108,7 +108,7 @@ accounts, err := keychain.GetGenericPasswordAccounts("MyService")
 // Should have 1 account == "gabriel"
 
 err := keychain.DeleteGenericPasswordItem("MyService", "gabriel")
-if err == keychain.ErrorItemNotFound {
+if errors.Is(err, keychain.ErrorItemNotFound) {
   // Not found
 }
 ```

bind/bind.go

@@ -3,6 +3,7 @@
 package bind
 
 import (
+	"errors"
 	"fmt"
 	"reflect"
 
@@ -65,7 +66,7 @@ func GenericPasswordTest(t Test, service string, accessGroup string) {
 
 	// Test dupe
 	err = keychain.AddItem(item)
-	if err != keychain.ErrorDuplicateItem {
+	if !errors.Is(err, keychain.ErrorDuplicateItem) {
 		t.Fail("Should error with duplicate item")
 	}
 
@@ -176,7 +177,7 @@ func GenericPasswordTest(t Test, service string, accessGroup string) {
 
 	// Test remove not found
 	err = keychain.DeleteItem(item)
-	if err != keychain.ErrorItemNotFound {
+	if !errors.Is(err, keychain.ErrorItemNotFound) {
 		t.Fail("Error should be not found")
 	}
 }

ios.go

@@ -10,13 +10,15 @@ package keychain
 */
 import "C"
 
-var AccessibleKey = attrKey(C.CFTypeRef(C.kSecAttrAccessible))
-var accessibleTypeRef = map[Accessible]C.CFTypeRef{
-	AccessibleWhenUnlocked:                   C.CFTypeRef(C.kSecAttrAccessibleWhenUnlocked),
-	AccessibleAfterFirstUnlock:               C.CFTypeRef(C.kSecAttrAccessibleAfterFirstUnlock),
-	AccessibleAlways:                         C.CFTypeRef(C.kSecAttrAccessibleAlways),
-	AccessibleWhenPasscodeSetThisDeviceOnly:  C.CFTypeRef(C.kSecAttrAccessibleWhenPasscodeSetThisDeviceOnly),
-	AccessibleWhenUnlockedThisDeviceOnly:     C.CFTypeRef(C.kSecAttrAccessibleWhenUnlockedThisDeviceOnly),
-	AccessibleAfterFirstUnlockThisDeviceOnly: C.CFTypeRef(C.kSecAttrAccessibleAfterFirstUnlockThisDeviceOnly),
-	AccessibleAccessibleAlwaysThisDeviceOnly: C.CFTypeRef(C.kSecAttrAccessibleAlwaysThisDeviceOnly),
-}
+var (
+	AccessibleKey     = attrKey(C.CFTypeRef(C.kSecAttrAccessible))
+	accessibleTypeRef = map[Accessible]C.CFTypeRef{
+		AccessibleWhenUnlocked:                   C.CFTypeRef(C.kSecAttrAccessibleWhenUnlocked),
+		AccessibleAfterFirstUnlock:               C.CFTypeRef(C.kSecAttrAccessibleAfterFirstUnlock),
+		AccessibleAlways:                         C.CFTypeRef(C.kSecAttrAccessibleAlways),
+		AccessibleWhenPasscodeSetThisDeviceOnly:  C.CFTypeRef(C.kSecAttrAccessibleWhenPasscodeSetThisDeviceOnly),
+		AccessibleWhenUnlockedThisDeviceOnly:     C.CFTypeRef(C.kSecAttrAccessibleWhenUnlockedThisDeviceOnly),
+		AccessibleAfterFirstUnlockThisDeviceOnly: C.CFTypeRef(C.kSecAttrAccessibleAfterFirstUnlockThisDeviceOnly),
+		AccessibleAccessibleAlwaysThisDeviceOnly: C.CFTypeRef(C.kSecAttrAccessibleAlwaysThisDeviceOnly),
+	}
+)

secretservice/secretservice.go

@@ -339,6 +339,11 @@ func (p PromptDismissedError) Error() string {
 	return p.err.Error()
 }
 
+// Unwrap
+func (p PromptDismissedError) Unwrap() error {
+	return p.err
+}
+
 // PromptAndWait is NOT thread-safe.
 func (s *SecretService) PromptAndWait(prompt dbus.ObjectPath) (paths *dbus.Variant, err error) {
 	if prompt == NullPrompt {

secretservice/secretservice_test.go

@@ -56,7 +56,6 @@ func testKeyring(t *testing.T, mode AuthenticationMode) {
 
 	err = srv.LockItems([]dbus.ObjectPath{collection})
 	require.NoError(t, err)
-
 }
 
 func TestGetAll(t *testing.T) {