Merge branch 'release/0.7.1'

Author: ledermann

.github/workflows/push.yml

@@ -72,14 +72,14 @@ jobs:
             type=semver,pattern={{major}}
             type=raw,value=latest,enable=${{ github.ref == format('refs/heads/{0}', 'main') }}
 
-      - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v3
-
       - name: Set up QEMU
         uses: docker/setup-qemu-action@v3
         with:
           platforms: linux/amd64,linux/arm64,linux/arm/v7
 
+      - name: Set up Docker Buildx
+        uses: docker/setup-buildx-action@v3
+
       - name: Login to GitHub Container Registry
         uses: docker/login-action@v3
         with:

Gemfile.lock

@@ -13,11 +13,11 @@ GEM
     csv (3.3.5)
     diff-lcs (1.6.2)
     docile (1.4.1)
-    dotenv (3.1.8)
+    dotenv (3.2.0)
     hashdiff (1.2.1)
     influxdb-client (3.2.0)
       csv
-    json (2.16.0)
+    json (2.17.1)
     language_server-protocol (3.17.0.5)
     lint_roller (1.1.0)
     logger (1.7.0)

app/config.rb

@@ -3,7 +3,7 @@ def initialize(options = {})
     options.each { |key, value| instance_variable_set("@#{key}", value) }
 
     validate_url!(influx_url)
-    validate_interval!(forecast_interval)
+    validate_interval!(forecast_interval) unless forecast_provider == 'pvnode'
   end
 
   attr_reader :influx_schema,
@@ -125,7 +125,7 @@ def forecast_solar_settings_from_env
       {
         forecast_provider: ENV.fetch('FORECAST_PROVIDER', 'forecast.solar'),
         forecast_solar_configurations: all_configurations_from_env('FORECAST', ForecastSolarConfiguration, defaults),
-        forecast_interval: ENV.fetch('FORECAST_INTERVAL').to_i,
+        forecast_interval: ENV.fetch('FORECAST_INTERVAL', nil)&.to_i,
         forecast_solar_apikey: ENV.fetch('FORECAST_SOLAR_APIKEY', nil),
       }
     end

spec/config_spec.rb

@@ -36,6 +36,13 @@
         expect { described_class.new(invalid_options) }.to raise_error(Exception, /URL is invalid/)
       end
     end
+
+    context 'with pvnode provider' do
+      it 'does not require forecast_interval' do
+        pvnode_options = valid_options.merge(forecast_provider: 'pvnode', forecast_interval: nil)
+        expect { described_class.new(pvnode_options) }.not_to raise_error
+      end
+    end
   end
 
   describe 'forecast methods' do