[dotnet] [bidi] Simplify modules namespace (breaking change) (#15820)

Author: nvborisenko

dotnet/src/webdriver/BiDi/BiDi.cs

@@ -27,41 +27,41 @@ public class BiDi : IAsyncDisposable
 {
     private readonly Broker _broker;
 
-    private readonly Lazy<Modules.Session.SessionModule> _sessionModule;
-    private readonly Lazy<Modules.BrowsingContext.BrowsingContextModule> _browsingContextModule;
-    private readonly Lazy<Modules.Browser.BrowserModule> _browserModule;
-    private readonly Lazy<Modules.Network.NetworkModule> _networkModule;
-    private readonly Lazy<Modules.Input.InputModule> _inputModule;
-    private readonly Lazy<Modules.Script.ScriptModule> _scriptModule;
-    private readonly Lazy<Modules.Log.LogModule> _logModule;
-    private readonly Lazy<Modules.Storage.StorageModule> _storageModule;
+    private readonly Lazy<Session.SessionModule> _sessionModule;
+    private readonly Lazy<BrowsingContext.BrowsingContextModule> _browsingContextModule;
+    private readonly Lazy<Browser.BrowserModule> _browserModule;
+    private readonly Lazy<Network.NetworkModule> _networkModule;
+    private readonly Lazy<Input.InputModule> _inputModule;
+    private readonly Lazy<Script.ScriptModule> _scriptModule;
+    private readonly Lazy<Log.LogModule> _logModule;
+    private readonly Lazy<Storage.StorageModule> _storageModule;
 
     internal BiDi(string url)
     {
         var uri = new Uri(url);
 
         _broker = new Broker(this, uri);
 
-        _sessionModule = new Lazy<Modules.Session.SessionModule>(() => new Modules.Session.SessionModule(_broker));
-        _browsingContextModule = new Lazy<Modules.BrowsingContext.BrowsingContextModule>(() => new Modules.BrowsingContext.BrowsingContextModule(_broker));
-        _browserModule = new Lazy<Modules.Browser.BrowserModule>(() => new Modules.Browser.BrowserModule(_broker));
-        _networkModule = new Lazy<Modules.Network.NetworkModule>(() => new Modules.Network.NetworkModule(_broker));
-        _inputModule = new Lazy<Modules.Input.InputModule>(() => new Modules.Input.InputModule(_broker));
-        _scriptModule = new Lazy<Modules.Script.ScriptModule>(() => new Modules.Script.ScriptModule(_broker));
-        _logModule = new Lazy<Modules.Log.LogModule>(() => new Modules.Log.LogModule(_broker));
-        _storageModule = new Lazy<Modules.Storage.StorageModule>(() => new Modules.Storage.StorageModule(_broker));
+        _sessionModule = new Lazy<Session.SessionModule>(() => new Session.SessionModule(_broker));
+        _browsingContextModule = new Lazy<BrowsingContext.BrowsingContextModule>(() => new BrowsingContext.BrowsingContextModule(_broker));
+        _browserModule = new Lazy<Browser.BrowserModule>(() => new Browser.BrowserModule(_broker));
+        _networkModule = new Lazy<Network.NetworkModule>(() => new Network.NetworkModule(_broker));
+        _inputModule = new Lazy<Input.InputModule>(() => new Input.InputModule(_broker));
+        _scriptModule = new Lazy<Script.ScriptModule>(() => new Script.ScriptModule(_broker));
+        _logModule = new Lazy<Log.LogModule>(() => new Log.LogModule(_broker));
+        _storageModule = new Lazy<Storage.StorageModule>(() => new Storage.StorageModule(_broker));
     }
 
-    internal Modules.Session.SessionModule SessionModule => _sessionModule.Value;
-    public Modules.BrowsingContext.BrowsingContextModule BrowsingContext => _browsingContextModule.Value;
-    public Modules.Browser.BrowserModule Browser => _browserModule.Value;
-    public Modules.Network.NetworkModule Network => _networkModule.Value;
-    internal Modules.Input.InputModule InputModule => _inputModule.Value;
-    public Modules.Script.ScriptModule Script => _scriptModule.Value;
-    public Modules.Log.LogModule Log => _logModule.Value;
-    public Modules.Storage.StorageModule Storage => _storageModule.Value;
+    internal Session.SessionModule SessionModule => _sessionModule.Value;
+    public BrowsingContext.BrowsingContextModule BrowsingContext => _browsingContextModule.Value;
+    public Browser.BrowserModule Browser => _browserModule.Value;
+    public Network.NetworkModule Network => _networkModule.Value;
+    internal Input.InputModule InputModule => _inputModule.Value;
+    public Script.ScriptModule Script => _scriptModule.Value;
+    public Log.LogModule Log => _logModule.Value;
+    public Storage.StorageModule Storage => _storageModule.Value;
 
-    public Task<Modules.Session.StatusResult> StatusAsync()
+    public Task<Session.StatusResult> StatusAsync()
     {
         return SessionModule.StatusAsync();
     }
@@ -75,7 +75,7 @@ public static async Task<BiDi> ConnectAsync(string url)
         return bidi;
     }
 
-    public Task EndAsync(Modules.Session.EndOptions? options = null)
+    public Task EndAsync(Session.EndOptions? options = null)
     {
         return SessionModule.EndAsync(options);
     }

dotnet/src/webdriver/BiDi/Modules/Browser/BrowserModule.cs renamed to dotnet/src/webdriver/BiDi/Browser/BrowserModule.cs

@@ -20,7 +20,7 @@
 using System.Threading.Tasks;
 using OpenQA.Selenium.BiDi.Communication;
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 public sealed class BrowserModule(Broker broker) : Module(broker)
 {

dotnet/src/webdriver/BiDi/Modules/Browser/ClientWindow.cs renamed to dotnet/src/webdriver/BiDi/Browser/ClientWindow.cs

@@ -17,7 +17,7 @@
 // under the License.
 // </copyright>
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 public record ClientWindow
 {

dotnet/src/webdriver/BiDi/Modules/Browser/ClientWindowInfo.cs renamed to dotnet/src/webdriver/BiDi/Browser/ClientWindowInfo.cs

@@ -17,7 +17,7 @@
 // under the License.
 // </copyright>
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 public record ClientWindowInfo(bool Active, ClientWindow ClientWindow, ClientWindowState State, int Height, int Width, int X, int Y);
 

dotnet/src/webdriver/BiDi/Modules/Browser/CloseCommand.cs renamed to dotnet/src/webdriver/BiDi/Browser/CloseCommand.cs

@@ -19,7 +19,7 @@
 
 using OpenQA.Selenium.BiDi.Communication;
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 internal class CloseCommand()
     : Command<CommandParameters, EmptyResult>(CommandParameters.Empty, "browser.close");

dotnet/src/webdriver/BiDi/Modules/Browser/CreateUserContextCommand.cs renamed to dotnet/src/webdriver/BiDi/Browser/CreateUserContextCommand.cs

@@ -19,7 +19,7 @@
 
 using OpenQA.Selenium.BiDi.Communication;
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 internal class CreateUserContextCommand(CreateUserContextCommandParameters @params)
     : Command<CreateUserContextCommandParameters, UserContextInfo>(@params, "browser.createUserContext");

dotnet/src/webdriver/BiDi/Modules/Browser/GetClientWindowsCommand.cs renamed to dotnet/src/webdriver/BiDi/Browser/GetClientWindowsCommand.cs

@@ -21,7 +21,7 @@
 using System.Collections;
 using System.Collections.Generic;
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 internal class GetClientWindowsCommand()
     : Command<CommandParameters, GetClientWindowsResult>(CommandParameters.Empty, "browser.getClientWindows");

dotnet/src/webdriver/BiDi/Modules/Browser/GetUserContextsCommand.cs renamed to dotnet/src/webdriver/BiDi/Browser/GetUserContextsCommand.cs

@@ -21,7 +21,7 @@
 using System.Collections;
 using System.Collections.Generic;
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 internal class GetUserContextsCommand()
     : Command<CommandParameters, GetUserContextsResult>(CommandParameters.Empty, "browser.getUserContexts");

dotnet/src/webdriver/BiDi/Modules/Browser/RemoveUserContextCommand.cs renamed to dotnet/src/webdriver/BiDi/Browser/RemoveUserContextCommand.cs

@@ -19,7 +19,7 @@
 
 using OpenQA.Selenium.BiDi.Communication;
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 internal class RemoveUserContextCommand(RemoveUserContextCommandParameters @params)
     : Command<RemoveUserContextCommandParameters, EmptyResult>(@params, "browser.removeUserContext");

dotnet/src/webdriver/BiDi/Modules/Browser/UserContext.cs renamed to dotnet/src/webdriver/BiDi/Browser/UserContext.cs

@@ -20,7 +20,7 @@
 using System;
 using System.Threading.Tasks;
 
-namespace OpenQA.Selenium.BiDi.Modules.Browser;
+namespace OpenQA.Selenium.BiDi.Browser;
 
 public class UserContext : IAsyncDisposable
 {