feat!: Refactor ownership pallet to transfer ownership of spaces, posts and domains

integration-tests/Cargo.toml

@@ -22,13 +22,14 @@ std = [
     'pallet-timestamp/std',
     'frame-support/std',
     'frame-system/std',
+    'pallet-domains/std',
     'pallet-permissions/std',
     'pallet-posts/std',
     'pallet-profiles/std',
     'pallet-reactions/std',
     'pallet-roles/std',
     'pallet-space-follows/std',
-    'pallet-space-ownership/std',
+    'pallet-ownership/std',
     'pallet-spaces/std',
     'subsocial-support/std',
 ]
@@ -47,13 +48,14 @@ sp-runtime = { git = 'https://github.com/paritytech/substrate', branch = 'polkad
 sp-std = { git = 'https://github.com/paritytech/substrate', branch = 'polkadot-v0.9.40', default-features = false }
 
 [dev-dependencies]
+pallet-domains = { path = '../pallets/domains', default-features = false }
 pallet-permissions = { path = '../pallets/permissions', default-features = false }
 pallet-posts = { path = '../pallets/posts', default-features = false }
 pallet-profiles = { path = '../pallets/profiles', default-features = false }
 pallet-reactions = { path = '../pallets/reactions', default-features = false }
 pallet-roles = { path = '../pallets/roles', default-features = false }
 pallet-space-follows = { path = '../pallets/space-follows', default-features = false }
-pallet-space-ownership = { path = '../pallets/space-ownership', default-features = false }
+pallet-ownership = { path = '../pallets/space-ownership', default-features = false }
 pallet-spaces = { path = '../pallets/spaces', default-features = false }
 subsocial-support = { path = '../pallets/support', default-features = false }
 

integration-tests/src/mock.rs

@@ -48,8 +48,9 @@ frame_support::construct_runtime!(
             Reactions: pallet_reactions,
             Roles: pallet_roles,
             SpaceFollows: pallet_space_follows,
-            SpaceOwnership: pallet_space_ownership,
+            SpaceOwnership: pallet_ownership,
             Spaces: pallet_spaces,
+            Domains: pallet_domains,
         }
     );
 
@@ -160,10 +161,15 @@ impl pallet_space_follows::Config for TestRuntime {
     type WeightInfo = pallet_space_follows::weights::SubstrateWeight<TestRuntime>;
 }
 
-impl pallet_space_ownership::Config for TestRuntime {
+impl pallet_ownership::Config for TestRuntime {
     type RuntimeEvent = RuntimeEvent;
     type ProfileManager = Profiles;
+    type SpacesInterface = Spaces;
+    type SpacePermissionsProvider = Spaces;
     type CreatorStakingProvider = ();
+    type DomainsProvider = Domains;
+    type PostsProvider = Posts;
+    type Currency = Balances;
     type WeightInfo = ();
 }
 
@@ -177,6 +183,30 @@ impl pallet_spaces::Config for TestRuntime {
     type WeightInfo = ();
 }
 
+parameter_types! {
+    pub const RegistrationPeriodLimit: BlockNumber = 100;
+    pub const BaseDomainDeposit: u64 = 10;
+    pub const OuterValueByteDeposit: u64 = 1;
+    pub const InitialPaymentBeneficiary: AccountId = ACCOUNT1;
+    pub InitialPricesConfig: pallet_domains::types::PricesConfigVec<TestRuntime> = vec![(1, 100)];
+}
+
+impl pallet_domains::Config for TestRuntime {
+    type RuntimeEvent = RuntimeEvent;
+    type Currency = Balances;
+    type MinDomainLength = ConstU32<1>;
+    type MaxDomainLength = ConstU32<64>;
+    type MaxDomainsPerAccount = ConstU32<10>;
+    type DomainsInsertLimit = ConstU32<10>;
+    type RegistrationPeriodLimit = RegistrationPeriodLimit;
+    type MaxOuterValueLength = ConstU32<64>;
+    type BaseDomainDeposit = BaseDomainDeposit;
+    type OuterValueByteDeposit = OuterValueByteDeposit;
+    type InitialPaymentBeneficiary = InitialPaymentBeneficiary;
+    type InitialPricesConfig = InitialPricesConfig;
+    type WeightInfo = ();
+}
+
 pub(crate) type AccountId = u64;
 pub(crate) type BlockNumber = u64;
 

integration-tests/src/tests/space_ownership.rs

@@ -7,7 +7,7 @@
 use frame_support::{assert_ok, assert_noop};
 use sp_runtime::traits::Zero;
 
-use pallet_space_ownership::Error as SpaceOwnershipError;
+use pallet_ownership::{EntityWithOwnership, Error as SpaceOwnershipError};
 use pallet_spaces::Error as SpacesError;
 
 use crate::mock::*;
@@ -20,7 +20,7 @@ fn transfer_space_ownership_should_work() {
         assert_ok!(_transfer_default_space_ownership()); // Transfer SpaceId 1 owned by ACCOUNT1 to ACCOUNT2
 
         assert_eq!(
-            SpaceOwnership::pending_space_owner(SPACE1).unwrap(),
+            SpaceOwnership::pending_ownership_transfer(EntityWithOwnership::Space(SPACE1)).unwrap(),
             ACCOUNT2
         );
     });
@@ -70,7 +70,7 @@ fn accept_pending_ownership_should_work() {
         assert_eq!(space.owner, ACCOUNT2);
 
         // Check that pending storage is cleared:
-        assert!(SpaceOwnership::pending_space_owner(SPACE1).is_none());
+        assert!(SpaceOwnership::pending_ownership_transfer(EntityWithOwnership::Space(SPACE1)).is_none());
 
         assert!(Balances::reserved_balance(ACCOUNT1).is_zero());
 
@@ -96,7 +96,7 @@ fn accept_pending_ownership_should_fail_when_no_pending_transfer_for_space() {
     ExtBuilder::build_with_space().execute_with(|| {
         assert_noop!(
             _accept_default_pending_ownership(),
-            SpaceOwnershipError::<TestRuntime>::NoPendingTransferOnSpace
+            SpaceOwnershipError::<TestRuntime>::NoPendingTransfer
         );
     });
 }
@@ -108,7 +108,7 @@ fn accept_pending_ownership_should_fail_if_origin_is_already_an_owner() {
 
         assert_noop!(
             _accept_pending_ownership(Some(RuntimeOrigin::signed(ACCOUNT1)), None),
-            SpaceOwnershipError::<TestRuntime>::AlreadyASpaceOwner
+            SpaceOwnershipError::<TestRuntime>::NotAllowedToAcceptOwnershipTransfer,
         );
     });
 }
@@ -137,7 +137,7 @@ fn reject_pending_ownership_should_work() {
         assert_eq!(space.owner, ACCOUNT1);
 
         // Check whether storage state is correct
-        assert!(SpaceOwnership::pending_space_owner(SPACE1).is_none());
+        assert!(SpaceOwnership::pending_ownership_transfer(EntityWithOwnership::Space(SPACE1)).is_none());
     });
 }
 
@@ -153,7 +153,7 @@ fn reject_pending_ownership_should_work_when_proposal_rejected_by_current_space_
         assert_eq!(space.owner, ACCOUNT1);
 
         // Check whether storage state is correct
-        assert!(SpaceOwnership::pending_space_owner(SPACE1).is_none());
+        assert!(SpaceOwnership::pending_ownership_transfer(EntityWithOwnership::Space(SPACE1)).is_none());
     });
 }
 
@@ -172,7 +172,7 @@ fn reject_pending_ownership_should_fail_when_no_pending_transfer_on_space() {
     ExtBuilder::build_with_space().execute_with(|| {
         assert_noop!(
             _reject_default_pending_ownership(),
-            SpaceOwnershipError::<TestRuntime>::NoPendingTransferOnSpace
+            SpaceOwnershipError::<TestRuntime>::NoPendingTransfer
         ); // Rejecting a transfer from ACCOUNT2
     });
 }

integration-tests/src/utils/space_ownership_utils.rs

@@ -5,6 +5,7 @@
 // Full license is available at https://github.com/dappforce/subsocial-parachain/blob/main/LICENSE
 
 use frame_support::pallet_prelude::*;
+use pallet_ownership::EntityWithOwnership;
 
 use subsocial_support::SpaceId;
 
@@ -20,9 +21,9 @@ pub(crate) fn _transfer_space_ownership(
     space_id: Option<SpaceId>,
     transfer_to: Option<AccountId>,
 ) -> DispatchResult {
-    SpaceOwnership::transfer_space_ownership(
+    SpaceOwnership::transfer_ownership(
         origin.unwrap_or_else(|| RuntimeOrigin::signed(ACCOUNT1)),
-        space_id.unwrap_or(SPACE1),
+        space_id.map_or(EntityWithOwnership::Space(SPACE1), |id| EntityWithOwnership::Space(id)),
         transfer_to.unwrap_or(ACCOUNT2),
     )
 }
@@ -34,7 +35,7 @@ pub(crate) fn _accept_default_pending_ownership() -> DispatchResult {
 pub(crate) fn _accept_pending_ownership(origin: Option<RuntimeOrigin>, space_id: Option<SpaceId>) -> DispatchResult {
     SpaceOwnership::accept_pending_ownership(
         origin.unwrap_or_else(|| RuntimeOrigin::signed(ACCOUNT2)),
-        space_id.unwrap_or(SPACE1),
+        space_id.map_or(EntityWithOwnership::Space(SPACE1), |id| EntityWithOwnership::Space(id)),
     )
 }
 
@@ -49,6 +50,6 @@ pub(crate) fn _reject_default_pending_ownership_by_current_owner() -> DispatchRe
 pub(crate) fn _reject_pending_ownership(origin: Option<RuntimeOrigin>, space_id: Option<SpaceId>) -> DispatchResult {
     SpaceOwnership::reject_pending_ownership(
         origin.unwrap_or_else(|| RuntimeOrigin::signed(ACCOUNT2)),
-        space_id.unwrap_or(SPACE1),
+        space_id.map_or(EntityWithOwnership::Space(SPACE1), |id| EntityWithOwnership::Space(id)),
     )
 }

pallets/creator-staking/src/tests/mock.rs

@@ -160,6 +160,8 @@ mock! {
 
     impl SpacesInterface<AccountId, SpaceId> for Spaces {
         fn get_space_owner(_space_id: SpaceId) -> Result<AccountId, DispatchError>;
+
+        fn update_space_owner(_space_id: SpaceId, _new_owner: AccountId) -> DispatchResult;
 
         fn create_space(_owner: &AccountId, _content: Content) -> Result<SpaceId, DispatchError>;
     }

pallets/domains/Cargo.toml

@@ -33,7 +33,10 @@ sp-core = { git = "https://github.com/paritytech/substrate", branch = "polkadot-
 
 [features]
 default = ["std"]
-runtime-benchmarks = ["frame-benchmarking"]
+runtime-benchmarks = [
+    "frame-benchmarking/runtime-benchmarks",
+    "subsocial-support/runtime-benchmarks",
+]
 std = [
     "codec/std",
     "scale-info/std",