diff --git a/include/fmt/ranges.h b/include/fmt/ranges.h
index 0b6ec900..3ea41c8c 100644
--- a/include/fmt/ranges.h
+++ b/include/fmt/ranges.h
@@ -155,7 +155,9 @@ template <typename T>
 struct has_mutable_begin_end<
     T, void_t<decltype(detail::range_begin(std::declval<T>())),
               decltype(detail::range_end(std::declval<T>())),
-              enable_if_t<std::is_copy_constructible<T>::value>>>
+              // the extra int here is because older versions of MSVC don't
+              // SFINAE properly unless there are distinct types
+              int>>
     : std::true_type {};
 
 template <typename T>
diff --git a/test/ranges-test.cc b/test/ranges-test.cc
index fa46fc41..8cafeec4 100644
--- a/test/ranges-test.cc
+++ b/test/ranges-test.cc
@@ -193,7 +193,7 @@ template <typename T> class noncopyable_range {
   std::vector<T> vec;
 
  public:
-  using const_iterator = typename ::std::vector<T>::const_iterator;
+  using iterator = typename ::std::vector<T>::iterator;
 
   template <typename... Args>
   explicit noncopyable_range(Args&&... args)
@@ -202,8 +202,8 @@ template <typename T> class noncopyable_range {
   noncopyable_range(noncopyable_range const&) = delete;
   noncopyable_range(noncopyable_range&) = delete;
 
-  const_iterator begin() const { return vec.begin(); }
-  const_iterator end() const { return vec.end(); }
+  iterator begin() { return vec.begin(); }
+  iterator end() { return vec.end(); }
 };
 
 TEST(ranges_test, range) {