Add CLI support for time_frame arrays in metrics

What does this MR do and why?

Related to #478008 (closed)

After !167201 (merged) has been deployed & tested on production, we now know that metrics with time_frame array are working.

Now, we add official support for them, by making the CLI use them as the default & adding them to our documentation.

This also updates all of the tests that were previously creating a separate metric definition for each time_frame.

References

Please include cross links to any resources that are relevant to this MR This will give reviewers and future readers helpful context to give an efficient review of the changes introduced.

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before	After

How to set up and validate locally

To test this, run the internal events CLI:

ruby scripts/internal_events/cli.rb

And try to create new metric definition with time_frame array. Then, verify that the new metrics are working, using the Manual testing in GDK commands listed by the CLI when using the view usage command of the CLI.

spec/scripts/internal_events/cli/helpers/metric_options_spec.rb

@@ -41,7 +41,7 @@
   context 'when option is for a supported and not yet defined metric' do
     it 'highlights key words in the name' do
       expect(option.formatted).to eq({
-        name: "<cyan>time frame</cyan> count of <cyan>unique users</cyan> who triggered a list of events",
+        name: "<cyan>Time frame</cyan> count of <cyan>unique users</cyan> who triggered a list of events",
         value: metrics
       })
     end
@@ -51,7 +51,7 @@
 
       it 'highlights key words in the name' do
         expect(option.formatted).to eq({
-          name: "<cyan>time frame</cyan> count of <cyan>unique users</cyan> who triggered a list of events " \
+          name: "<cyan>Time frame</cyan> count of <cyan>unique users</cyan> who triggered a list of events " \
             "<cyan>where label/prop/anything</cyan> is...",
           value: metrics
         })
@@ -64,7 +64,7 @@
 
     it 'formats the option as disabled' do
       expect(option.formatted).to eq({
-        name: "<bright_black>time frame count of unique users who triggered a list of events</bright_black>",
+        name: "<bright_black>Time frame count of unique users who triggered a list of events</bright_black>",
         value: metrics,
         disabled: "<bold><bright_black>(already defined)</bright_black></bold>"
       })
@@ -75,7 +75,7 @@
 
       it 'highlights key words in the name' do
         expect(option.formatted).to eq({
-          name: "<bright_black>time frame count of unique users who triggered " \
+          name: "<bright_black>Time frame count of unique users who triggered " \
             "a list of events where filtered</bright_black>",
           value: metrics,
           disabled: "<bold><bright_black>(already defined)</bright_black></bold>"
@@ -89,7 +89,7 @@
 
     it 'formats the option as disabled' do
       expect(option.formatted).to eq({
-        name: "<bright_black>time frame count of unique users who triggered a list of events</bright_black>",
+        name: "<bright_black>Time frame count of unique users who triggered a list of events</bright_black>",
         value: metrics,
         disabled: "<bold><bright_black>(user unavailable)</bright_black></bold>"
       })
@@ -100,7 +100,7 @@
 
       it 'highlights key words in the name' do
         expect(option.formatted).to eq({
-          name: "<bright_black>time frame count of unique users who triggered " \
+          name: "<bright_black>Time frame count of unique users who triggered " \
             "a list of events where filtered</bright_black>",
           value: metrics,
           disabled: "<bold><bright_black>(user unavailable)</bright_black></bold>"
@@ -114,7 +114,7 @@
 
     it 'highlights key words in the name' do
       expect(option.formatted).to eq({
-        name: "<cyan>time frame</cyan> count of <cyan>unique values for 'label'</cyan> " \
+        name: "<cyan>Time frame</cyan> count of <cyan>unique values for 'label'</cyan> " \
           "from a list of events occurrences",
         value: metrics
       })
@@ -126,7 +126,7 @@
 
     it 'highlights key words in the name' do
       expect(option.formatted).to eq({
-        name: "<cyan>time frame</cyan> count of a list of events occurrences",
+        name: "<cyan>Time frame</cyan> count of a list of events occurrences",
         value: metrics
       })
     end
@@ -144,7 +144,7 @@
 
     it 'highlights key words in the name' do
       expect(option.formatted).to eq({
-        name: "<cyan>time frame 1/time frame 2</cyan> count of <cyan>unique users</cyan> " \
+        name: "<cyan>Time frame 1/Time frame 2</cyan> count of <cyan>unique users</cyan> " \
           "who triggered a list of events",
         value: metrics
       })