Repository: getmoto/moto (python)

Issue:
[RDS - Moto 5.1.3] stop_db_instance does not respect DBSnapshotIdentifier as manual snapshots
When calling stop_db_instance, you can provide a DBSnapshotIdentifier to create a manual snapshot as part of the db stop action. However, moto incorrectly tags this snapshot as an "automatic" snapshot instead of a manual one.

Example Code: 
```python

#stop db instance (should create a manual snapshot)
rds_client.stop_db_instance(DBInstanceIdentifier="myInstance", DBSnapshotIdentifier="mySnapshot")


# returns empty (should return a snapshot)
rds_client.describe_db_snapshots(
            DBInstanceIdentifier=rds_instance, SnapshotType="manual"
        )
```

running the describe_db_snapshots command without the SnapshotType filter does return the snapshot, but it is incorrectly tagged as "automatic" 

prio to moto 5.1, my automated tests worked fine because they (unknowingly) leveraged the bug that was fixed in the commit mentioned in #8658. however, now there is both an automated snapshot along with my manual snapshot that I am trying to test for, so my tests that were previously looking for the presense/absense of a snapshot no longer work, and this bug with not correctly tagging the snapshot as a manual snapshot prevents me from being able to update the tests to correctly query for manual snapshots.




Pull request description:
RDS: snapshot type should be `manual` when stopping a DBInstance
Fixes #8779 

Patch:
diff --git a/moto/rds/models.py b/moto/rds/models.py
index f3621b1df..3a291b6d6 100644
--- a/moto/rds/models.py
+++ b/moto/rds/models.py
@@ -2690,7 +2690,7 @@ class RDSBackend(BaseBackend):
         if database.status != "available":
             raise InvalidDBInstanceStateError(db_instance_identifier, "stop")
         if db_snapshot_identifier:
-            self.create_auto_snapshot(db_instance_identifier, db_snapshot_identifier)
+            self.create_db_snapshot(db_instance_identifier, db_snapshot_identifier)
         database.status = "stopped"
         return database
 


Test patch:
diff --git a/tests/test_rds/test_rds.py b/tests/test_rds/test_rds.py
index 78dd005c4..7fdf0446c 100644
--- a/tests/test_rds/test_rds.py
+++ b/tests/test_rds/test_rds.py
@@ -289,6 +289,7 @@ def test_start_database(client):
     assert response["DBInstance"]["DBInstanceStatus"] == "stopped"
     response = client.describe_db_snapshots(DBSnapshotIdentifier="rocky4570-rds-snap")
     assert response["DBSnapshots"][0]["DBSnapshotIdentifier"] == "rocky4570-rds-snap"
+    assert response["DBSnapshots"][0]["SnapshotType"] == "manual"
     response = client.start_db_instance(
         DBInstanceIdentifier=mydb["DBInstanceIdentifier"]
     )