Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 38e2fbdc authored by Paul Duffin's avatar Paul Duffin Committed by Gerrit Code Review
Browse files

Merge changes If673d482,I98b575b8

* changes:
  Remove reference to obsolete BuildSnapshot()
  Correct order of parameters to AssertDeepEquals
parents 4779327c 425b0eac
Loading
Loading
Loading
Loading
+1 −7
Original line number Diff line number Diff line
@@ -331,13 +331,7 @@ type SdkMemberType interface {

	// Add a prebuilt module that the sdk will populate.
	//
	// Returning nil from this will cause the sdk module type to use the deprecated BuildSnapshot
	// method to build the snapshot. That method is deprecated because it requires the SdkMemberType
	// implementation to do all the word.
	//
	// Otherwise, returning a non-nil value from this will cause the sdk module type to do the
	// majority of the work to generate the snapshot. The sdk module code generates the snapshot
	// as follows:
	// The sdk module code generates the snapshot as follows:
	//
	// * A properties struct of type SdkMemberProperties is created for each variant and
	//   populated with information from the variant by calling PopulateFromVariant(SdkAware)
+9 −6
Original line number Diff line number Diff line
@@ -283,7 +283,7 @@ func TestCommonValueOptimization(t *testing.T) {
	extractor.extractCommonProperties(common, structs)

	h := TestHelper{t}
	h.AssertDeepEquals("common properties not correct", common,
	h.AssertDeepEquals("common properties not correct",
		&testPropertiesStruct{
			private:     "",
			Public_Kept: "",
@@ -297,9 +297,10 @@ func TestCommonValueOptimization(t *testing.T) {
				S_Embedded_Common:    "embedded_common",
				S_Embedded_Different: "",
			},
		})
		},
		common)

	h.AssertDeepEquals("updated properties[0] not correct", structs[0],
	h.AssertDeepEquals("updated properties[0] not correct",
		&testPropertiesStruct{
			private:     "common",
			Public_Kept: "common",
@@ -313,9 +314,10 @@ func TestCommonValueOptimization(t *testing.T) {
				S_Embedded_Common:    "",
				S_Embedded_Different: "embedded_upper",
			},
		})
		},
		structs[0])

	h.AssertDeepEquals("updated properties[1] not correct", structs[1],
	h.AssertDeepEquals("updated properties[1] not correct",
		&testPropertiesStruct{
			private:     "common",
			Public_Kept: "common",
@@ -329,5 +331,6 @@ func TestCommonValueOptimization(t *testing.T) {
				S_Embedded_Common:    "",
				S_Embedded_Different: "embedded_lower",
			},
		})
		},
		structs[1])
}