Tu variable está undefined
porque está dentro de un local scope que en este caso sería el then
si quieres acceder a esa variable desde un scope global no podrás porque solo está disponible dentro de ese scope y de los scope más anidados, no sé si logro explicarme.
Mete tu condicional dentro del then
y ya está
//CODIGO PARA LLAMAR FECHA Y HORA DE LA BASE DE DATOS
db.collection("citas_registradas").where("fecha", "==", fecha).where("hora", "==", hora)
.get()
.then((querySnapshot) => {
var fechaV;
var horaV;
querySnapshot.forEach((doc) => {
fechaV = fecha;
horaV = hora;
});
//validacion si la fecha ingresada es igual a la fecha que retorna la base de datos y la hora es igual a la hora
//que retorna la base de datos, es por que la cita está llena.
if (fecha == fechaV && hora == horaV) {
Swal.fire("Oops!", "La cita ya está llena, por favor, busque una hora libre y asigne la cita.", "error");
//si la fecha es diferente a la que retorna la base de datos y la hora igual, o la fecha es igual a la de la base
//de datos pero la hora es diferente, registra la cita.
} else if (fecha != fechaV && hora != horaV || fecha == fechaV && hora != horaV) {
Swal.fire("¡Genial!", "La cita está libre.", "success");
}