Adds parameter completer for -Period in Reports module (#1956)

* Add argument completer for -Period parameter.

* Fix missing -OutFile parameter

Author: peombwa

src/Reports/Reports.md

@@ -17,6 +17,13 @@ require:
 
 ``` yaml
 directive:
+  - where:
+      parameter-name: Period
+    set:
+      completer:
+        name: Period Completer
+        description: Gets the list of Period values.
+        script: "'D7', 'D30', 'D90', 'D180'"
 # Remove invalid paths.
   - remove-path-by-operation: auditLogs\.auditLogRoot.*|reports.reportRoot.*|(auditLogs|reports)_(Create|Delete|Update).*
 # Remove cmdlets

src/Reports/v1.0/test/Get-MgReportEmailActivityCount.Tests.ps1

@@ -0,0 +1,29 @@
+# ------------------------------------------------------------------------------
+# Copyright (c) Microsoft Corporation. All Rights Reserved. Licensed under the MIT License. See License in the project root for license information.
+# ------------------------------------------------------------------------------
+Describe "Get-MgReportEmailActivityCount" {
+    BeforeAll {
+        $GetMgReportEmailActivityCount = Get-Command Get-MgReportEmailActivityCount
+    }
+
+    It "Should support minimum set of parameter sets" {
+        $GetMgReportEmailActivityCount.ParameterSets.Name | Should -BeIn @("Get", "GetViaIdentity")
+        $GetMgReportEmailActivityCount.Visibility | Should -Be "Public"
+        $GetMgReportEmailActivityCount.CommandType | Should -Be "Function"
+    }
+
+    It "Should return a list of applications by default" {
+        $GetMgReportEmailActivityCount.OutputType | Should -Be "System.Boolean"
+        $GetMgReportEmailActivityCount.DefaultParameterSet | Should -Be "Get"
+    }
+
+    It 'Should have Get parameterSet' {
+        $GetParameterSet = $GetMgReportEmailActivityCount.ParameterSets | Where-Object Name -eq "Get"
+        $GetParameterSet.Parameters.Name | Should -Contain OutFile
+        $GetParameterSet.Parameters.Name | Should -Contain Period
+    }
+
+    It '-Period parameter should have an argument completer' {
+        $GetMgReportEmailActivityCount.Parameters.Period.Attributes.ScriptBlock | Should -be "'D7', 'D30', 'D90', 'D180'"
+    }
+}

src/readme.graph.md

@@ -492,6 +492,11 @@ directive:
   - from: openapi-document
     where: $..paths.*[?(/(.*_GetCount)/gmi.exec(@.operationId))]..parameters[?(@.name === "ConsistencyLevel")]
     transform: $['required'] = true
+# Fix binary response definition for AutoREST to generate -OutFile parameter.
+  - from: openapi-document
+    where: $..paths..responses['2XX'].content['application/octet-stream'].schema
+    transform: >-
+      if ($.type === 'object') { $['format'] = "binary" }
 # Modify generated .json.cs model classes.
   - from: source-file-csharp
     where: $