Update location.proto package name

This commit updates the package name of `location.proto` from
`frequenz.api.common.v1.location` to `frequenz.api.common.v1`.

This sets package name of this proto file to its parent directory name,
which seems to be a standard practice.

Signed-off-by: Tiyash Basu <[email protected]>

Author: tiyash-basu-frequenz

RELEASE_NOTES.md

@@ -57,6 +57,8 @@
 
 - Added a message `Microgrid` to represent a microgrid.
 
+- Updated the package name of `location.proto` to `frequenz.api.common.v1`.
+
 ## New Features
 
 <!-- Here goes the main new features and examples or instructions on how to use them -->

proto/frequenz/api/common/v1/location.proto

@@ -7,7 +7,7 @@
 
 syntax = "proto3";
 
-package frequenz.api.common.v1.location;
+package frequenz.api.common.v1;
 
 // A pair of geographical co-ordinates, representing the location of a place.
 message Location {

proto/frequenz/api/common/v1/microgrid/microgrid.proto

@@ -54,7 +54,7 @@ message Microgrid {
   frequenz.api.common.v1.grid.DeliveryArea delivery_area = 4;
 
   // Physical location of the microgrid, in geographical co-ordinates.
-  frequenz.api.common.v1.location.Location location = 5;
+  frequenz.api.common.v1.Location location = 5;
 
   // The current status of the microgrid.
   MicrogridStatus status = 6;

pytests/test_common.py

@@ -166,3 +166,16 @@ def test_module_import_microgrid_components() -> None:
     from frequenz.api.common.v1.microgrid.components import transformer_pb2_grpc
 
     assert transformer_pb2_grpc is not None
+
+
+def test_module_import_location() -> None:
+    """Test that the modules can be imported."""
+    # pylint: disable=import-outside-toplevel
+    from frequenz.api.common.v1 import location_pb2
+
+    assert location_pb2 is not None
+
+    # pylint: disable=import-outside-toplevel
+    from frequenz.api.common.v1 import location_pb2_grpc
+
+    assert location_pb2_grpc is not None