Parcourir la source

rollback test state to before template variables were added

Erik Sundell il y a 7 ans
Parent
commit
243c4bfe29

+ 6 - 51
public/app/plugins/datasource/stackdriver/specs/query_aggregation_ctrl.test.ts

@@ -17,68 +17,27 @@ describe('StackdriverAggregationCtrl', () => {
             },
             {
               replace: s => s,
-              variables: [{ name: 'someVariable1' }, { name: 'someVariable2' }],
             }
           );
         });
 
         it('should populate all aggregate options except two', () => {
           ctrl.setAggOptions();
-          expect(ctrl.aggOptions.length).toBe(2);
-          const [templateVariableGroup, aggOptionsGroup] = ctrl.aggOptions;
-          expect(templateVariableGroup.options.length).toBe(2);
-          expect(aggOptionsGroup.options.length).toBe(11);
-          expect(aggOptionsGroup.options.map(o => o.value)).toEqual(
+          expect(ctrl.aggOptions.length).toBe(11);
+          expect(ctrl.aggOptions.map(o => o.value)).toEqual(
             expect['not'].arrayContaining(['REDUCE_COUNT_TRUE', 'REDUCE_COUNT_FALSE'])
           );
         });
 
         it('should populate all alignment options except two', () => {
           ctrl.setAlignOptions();
-          const [templateVariableGroup, alignOptionGroup] = ctrl.aggOptions;
-          expect(templateVariableGroup.options.length).toBe(2);
-          expect(alignOptionGroup.options.length).toBe(11);
-          expect(alignOptionGroup.options.map(o => o.value)).toEqual(
+          expect(ctrl.alignOptions.length).toBe(9);
+          expect(ctrl.alignOptions.map(o => o.value)).toEqual(
             expect['not'].arrayContaining(['REDUCE_COUNT_TRUE', 'REDUCE_COUNT_FALSE'])
           );
         });
       });
 
-      describe('and result is double and delta and no group by is used', () => {
-        beforeEach(async () => {
-          ctrl = new StackdriverAggregationCtrl(
-            {
-              $on: () => {},
-              target: {
-                valueType: 'DOUBLE',
-                metricKind: 'DELTA',
-                aggregation: { crossSeriesReducer: '', groupBys: [] },
-              },
-            },
-            {
-              replace: s => s,
-              variables: [{ name: 'someVariable1' }, { name: 'someVariable2' }],
-            }
-          );
-        });
-
-        it('should populate all alignment options except four', () => {
-          ctrl.setAlignOptions();
-          const [templateVariableGroup, alignOptionGroup] = ctrl.alignOptions;
-          expect(templateVariableGroup.options.length).toBe(2);
-          expect(alignOptionGroup.options.length).toBe(9);
-          expect(alignOptionGroup.options.map(o => o.value)).toEqual(
-            expect['not'].arrayContaining([
-              'ALIGN_NEXT_OLDER',
-              'ALIGN_INTERPOLATE',
-              'ALIGN_COUNT_TRUE',
-              'ALIGN_COUNT_FALSE',
-              'ALIGN_FRACTION_TRUE',
-            ])
-          );
-        });
-      });
-
       describe('and result is double and gauge and a group by is used', () => {
         beforeEach(async () => {
           ctrl = new StackdriverAggregationCtrl(
@@ -92,18 +51,14 @@ describe('StackdriverAggregationCtrl', () => {
             },
             {
               replace: s => s,
-              variables: [{ name: 'someVariable1' }],
             }
           );
         });
 
         it('should populate all aggregate options except three', () => {
           ctrl.setAggOptions();
-          const [templateVariableGroup, aggOptionsGroup] = ctrl.aggOptions;
-          expect(ctrl.aggOptions.length).toBe(2);
-          expect(templateVariableGroup.options.length).toBe(1);
-          expect(aggOptionsGroup.options.length).toBe(10);
-          expect(aggOptionsGroup.options.map(o => o.value)).toEqual(
+          expect(ctrl.aggOptions.length).toBe(10);
+          expect(ctrl.aggOptions.map(o => o.value)).toEqual(
             expect['not'].arrayContaining(['REDUCE_COUNT_TRUE', 'REDUCE_COUNT_FALSE', 'REDUCE_NONE'])
           );
         });