MaxACPower: fix decoration

[mfuchs1984]

@andig I had a look at #18943 and #18937 and did some debugging.
After a while, I figured out, that the test for api.MaxACPower in

evcc/core/site.go

Line 486 in 714438d

if m, ok := meter.(api.MaxACPower); ok {

always failed, although the interface was decorated.
After renaming the interface from MaxACPower to something not matching the api function name, in this case MaxACPowerGetter, it started to work. As a golang newbie, I am not sure exactly why.

[andig]

Das kann nicht das Problem sein: https://play.golang.com/p/81MisrDtabG

[mfuchs1984]

Ich verstehe es auch nicht, aber es ist ganz simpel nachzuvollziehen. Einfach ein Meter anlegen mit maxacpower (muss nicht in echt existieren) und einen Breakpoint in

evcc/core/site.go

Line 486 in 714438d

if m, ok := meter.(api.MaxACPower); ok {

setzen und warten, bis das entsprechende Meter dran ist. Ohne die Änderung schlägt der Test bei mir reproduzierbar fehl, mit der Änderung geht's reproduzierbar.

[andig]

However this test confirms that it's not working as expected:

func TestACPower(t *testing.T) {
	m, err := NewConfigurableFromConfig(context.TODO(), map[string]any{
		"maxacpower": 1000,
		"power": map[string]any{
			"source": "const",
			"value":  1000,
		},
	})
	require.NoError(t, err)
	_, ok := m.(api.MaxACPower)
	require.True(t, ok)
}

Let's find out why.

[andig]

See https://groups.google.com/g/golang-nuts/c/nOrO4mDW-7U

[mfuchs1984]

Habe das auch nur ausprobiert, weil das der einzige Unterschied war, den ich zu den funktionierenden Interfaces gefunden habe.

[andig]

pls run

make assets

to fix the build

[mfuchs1984]

Will do later today.

[mfuchs1984]

Done