Fix e2e test addon deprovisioning order

Signed-off-by: James Munnelly <james@munnelly.eu>

test/e2e/framework/framework.go

@@ -152,6 +152,17 @@ func (f *Framework) AfterEach() {
 
 	f.printAddonLogs()
 
+	if !f.Config.Cleanup {
+		return
+	}
+
+	for i := len(f.requiredAddons) - 1; i >= 0; i-- {
+		a := f.requiredAddons[i]
+		By("De-provisioning test-scoped addon")
+		err := a.Deprovision()
+		Expect(err).NotTo(HaveOccurred())
+	}
+
 	By("Deleting test namespace")
 	err := f.DeleteKubeNamespace(f.Namespace.Name)
 	Expect(err).NotTo(HaveOccurred())
@@ -209,14 +220,6 @@ func (f *Framework) RequireAddon(a addon.Addon) {
 		err = a.Provision()
 		Expect(err).NotTo(HaveOccurred())
 	})
-
-	AfterEach(func() {
-		if !f.Config.Cleanup {
-			return
-		}
-		err := a.Deprovision()
-		Expect(err).NotTo(HaveOccurred())
-	})
 }
 
 func (f *Framework) Helper() *helper.Helper {